Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dark theme: "Hide Search Matches" shown on mobile #117

Closed
hugovk opened this issue Mar 18, 2023 · 7 comments · Fixed by #124
Closed

Dark theme: "Hide Search Matches" shown on mobile #117

hugovk opened this issue Mar 18, 2023 · 7 comments · Fixed by #124
Labels

Comments

@hugovk
Copy link
Member

hugovk commented Mar 18, 2023

  1. Using the new dark theme (Dark theme #44) in mobile layout: https://docs.python.org/dev/
  2. Search for something (e.g. "turtle")

image

  1. Click a result

image

Actual result: "Hide Search Matches" is shown subscript to the right of the search box

image

Expected result: It's either shown more neatly, or, like desktop view, isn't shown at all:

image

cc @septatrix

@hugovk hugovk added the bug label Mar 18, 2023
@septatrix
Copy link
Contributor

Okay I found the culprit/fix to restore the previous behaviour (not shown at all). Do we want that or would it actually be desirable to show it?

@septatrix
Copy link
Contributor

Given that it is only ever added to either the mobile UI or the desktop UI I feel like it is probably less confusing to suppress it again, aye?

@rik
Copy link

rik commented Mar 21, 2023

Besides the UI glitch, I was actually glad to discover this "bug" as I'm not fond of matches being highlighted.

I wish it was available on the wide screen UI too.

@septatrix
Copy link
Contributor

The problem is that the toggle is only added to the #searchbox element of which there can be only one. So either it would not be visible in one of the two layouts or we would have to do some pretty crazy CSS shenanigans to reuse the same search box in the desktop and mobile layout.

@hugovk
Copy link
Member Author

hugovk commented Mar 25, 2023

I wish it was available on the wide screen UI too.

You can press esc to remove the highlights.


Given that it is only ever added to either the mobile UI or the desktop UI I feel like it is probably less confusing to suppress it again, aye?

Yeah, no strong opinion from me, if in doubt let's go for the previous behaviour. Thank you!

@hugovk
Copy link
Member Author

hugovk commented Apr 11, 2023

@septatrix Would you be able to restore the previous behaviour? For me, this is the only one of the reported issues I'd like fixed before deploying this more widely. And I've heard lots of good feedback about the dark theme!

septatrix added a commit to septatrix/python-docs-theme that referenced this issue Apr 12, 2023
@septatrix
Copy link
Contributor

septatrix commented Apr 12, 2023

@septatrix Would you be able to restore the previous behaviour? For me, this is the only one of the reported issues I'd like fixed before deploying this more widely. And I've heard lots of good feedback about the dark theme!

Done, #124 restores the previous behaviour

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants