forum

[web] Can't change thread page when searching with spaces [confirmed]

posted
Total Posts
42
Topic Starter
Timekiller
1) Enter search string using space, like "test build"
2) Click on any link that follows "Topic:", like "osu!stream Discussion", for a topic that has more than 1 page
3) Pages won't change when you click on their number. Moreso, "hilit" parameter of url becomes weirder and weirder each time you try to change the page.

Easily fixed by adding "?" symbol before parameter string, like
http://osu.ppy.sh/forum/t/58884/?hilit= ... d&start=45
instead of
http://osu.ppy.sh/forum/t/58884/hilit=t ... d&start=45
MillhioreF
SPOILER
http://osu.ppy.sh/forum/t/58884/hilit=test%2Bbuild%26amp%3Bamp%3Bamp%3Bamp%3Bamp%3Bamp%3Bamp%3Bstart%3D15%26amp%3Bamp%3Bamp%3Bamp%3Bamp%3Bamp%3Bstart%3D15%26amp%3Bamp%3Bamp%3Bamp%3Bamp%3Bstart%3D15%26amp%3Bamp%3Bamp%3Bamp%3Bstart%3D15%26amp%3Bamp%3Bamp%3Bstart%3D15%26amp%3Bamp%3Bstart%3D15%26amp%3Bstart%3D45&start=30
wow, that's... pretty bad D: confirming
My1_old
just fix it ppy that would be nic3
deadbeat
thats been around for a while i think. i think its cause the search is towards that page. you can fix this buy clicking on the thread topic name.
example
p/1785214/hilit=filter takes you to Index » osu! » Feature Requests » Allow to use different Approach Rates in map
clicking on Allow to use different Approach Rates in map takes you to
t/47941/start=0 and then you can swap pages again.
its kinda a round about way. but it does work. hope you can understand that x.x
Topic Starter
Timekiller
Well, but it cancels the search highlighting. Kind of defeating the purpose :3
...Or so I would like say, but it just hit me that highlighting does not work at all.
peppy
This should now be resolved :).
Topic Starter
Timekiller
Nope :D
t/76914?hilit=test+build
Page switcher links are still t/76914/hilit=test+build&start=15 , not t/76914?hilit=test+build&start=15 , so it's still the same problem :3

Also, is highlighting not supposed to work? span class="posthilit" is present on page, but it seems there's no css entry for it.
Topic Starter
Timekiller
The issue is still NOT resolved, fyi. Why would you move the topic?
peppy
I believe the issue was resolved because while your link above doesn't work, it is not longer possible to create such a malformed URL. If it is, please provide a link to the search page, rather than the resulting linked page.
Topic Starter
Timekiller
Topic Starter
Timekiller
Oh, come on, it's moved again?
MillhioreF
Ah yes, confirming; it's almost fixed, but not quite!

t/67679?hilit=bug+remains
t/67679/hilit=bug+remains&start=15

Note how the ? after the topic number changes to a / when you change the page. This one character is doing all the breaking...
peppy
Being moved isn't an issue (was automated); I still follow threads in this forum.
MillhioreF
This issue seems to have returned (see t/108560/)
Topic Starter
Timekiller

MillhioreF wrote:

This issue seems to have returned (see t/108560/)
I believe it was never actually fixed completely. Yet the topic was already moved to 'resolved' twice, so I just stopped bumping it :| I value my account too much to provoke dev staff with my grumbling :3
MillhioreF
I wasn't able to move it out of completed requests back then, either... but now I am! Let's see if we can get this fixed for good.
peppy
As the search highlighting wasn't even working, I have removed this from the URLs provided in search. It should also fix this issue.
TheVileOne
This still happens. It is not resolved.
peppy
Please provide the exact case it still fails. I can't replicate.
TheVileOne
peppy
I need to know how you got to that page, though. Can you provide each step in the process?
TheVileOne
I searched James training. I click on the link and it links to

t/4280?hilit=James+Training

Just standard old search with topic enabled.
MillhioreF
I go to the bottom of Gameplay and Rankings and search "james training" in the bottom box:



Then I can click on the title of any search result and it will result in the "hilit" parameter.



"Jump to post" works fine, however.
Topic Starter
Timekiller
By the way, you could make post highlighting work, if you want. Search terms are properly wrapped in
<span class="posthilit">
, so all you need to do is to modify stylesheets.
TheVileOne
This is still broken.
MillhioreF
The search highlighting still occurs if you click on the "Topic: <topicname>" in the upper bar of a search result. Moving out of Resolved yet again :o
kurufu
So this was brought up a year ago or something here
http://osu.ppy.sh/forum/t/78638?hilit=web+search

However it is broken again. If i were to go search and pull up a thread i get a link like
http://osu.ppy.sh/forum/t/132667/hilit=BAT+is+now+officially+managed
selecting next or jumping to a page with the numbers next to "next" gives me
http://osu.ppy.sh/forum/t/132667/hilit=BAT+is+now+officially+managed&start=15
which fails to go to the next/specified page. However
http://osu.ppy.sh/forum/t/132667/start=15
does take me to the correct page.

This fix would make it so much easier to search the forum before i make bug requests and we hate duplicate bug requests right? (completely disregarding the fact that the "hilit=XxxXXx" doesnt seem to add anything to my browsing experience other than inducing this bug)

Duplicatable on any multi page forum post you can find with search when browsing with chrome 26.0.1410.64 m

---EDIT---
It appears this only happens with the little search box at the bottom of a forum/thread (or at the top of a completed search). Because only from that little box does it start adding the "hilit=XxXxX" stuff. Using the Search button at the top of the page next to Settings does not add the "hilit=XxxXXx" stuff and so doesnt produce this bug.
MillhioreF
Bump! Been getting a couple duplicate topics about this recently.
MillhioreF
Duplicate of t/95686 - I don't blame you for missing it though, it was on page 3 of low priority. I've bumped it back up again to be more visible, since it's quite an annoyance IMO.
peppy
What part isn't working here?
XPJ38
There's a detailed post here: t/133253
MillhioreF
Highlight is working now (which is cool) but this bug persists!
theowest

XPJ38 wrote:

There's a detailed post here: t/133253
I'll just merge that then.
My1_old
@devs good Luck fixing
@Millhi thx for keeping this up.
theowest
but isn't this pretty low priority? I mean, not many use search and then try to go to the next page. Us maybe, but it's so extremely easy to resolve, just hit the title of the topic or erase the +search keywords in the url.
TheVileOne
It primarily affects us who actually are supposed to use search as it's intended to be used.

It's medium-low priority.
show more
Please sign in to reply.

New reply