-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
📘 DOC: Move ThemeToggleButton to fixed position on mobile #107
Comments
I will explore both and get back to you |
Ping @aFuzzyBear @tony-sull & @delucis on this, to decide whether this is something to explore/keep on the stack! |
when I did take a look at this the theme toggle changed and took hold else whr, we can revisit this for sure, since I can't see the dark mode on the mobile |
I don’t think it’s a high priority, but it is something we could address at some point. We do now show the “More” menu as part of the top table of contents on mobile, but don’t include the theme toggle. (I’m actually not 100% on giving the more menu such priority on mobile, but that’s a side quest we’ll ignore for now.) Real estate is at a bit of a premium on mobile. Overview of how a few other docs sites handle this:
If I were to tackle this, I’d probably do a more holistic re-organisation of stuff:
|
I like @delucis 's suggestions, and confirming @aFuzzyBear 's observation that I can't see the toggle on mobile. Since this original Issue, the More menu seems to have been moved to the top of the page, directly after what would normally be the Right Sidebar. So, if we choose not to envision a complete redesign at this point, can we confirm the immediate issues to address here are:
|
Would it make sense to to move the language picker into the menu drawer on mobile? That could give space for the theme toggle in the top header bar and let the language picker have the full screen width so it doesn't get cropped |
I personally feel like choosing the language you understand is probably a higher priority task than switching theme. If we hide the language switcher I think some users would likely assume the docs were only available in English. |
Nice work @Yan-Thomas! Love the screenshots and gathering feedback like this 💜 I think on small screens the theme toggle should definitely go inside the pop-up menu, but I’d put it at the bottom of the menu rather than at the top. I like how the mobile menu on the Vue site handles this for example (https://vuejs.org/). I’m also in favour of collapsing menu sections on mobile at some point so even if this is at the bottom, it would be a little less buried. I kind of like your medium screen solution too, so maybe the best solution could be:
There might also still be a grey area where the menu switches to the left sidebar but there’s no room in the top bar yet? In that case I’d show the toggle on the bottom of the left sidebar still for consistency with the mobile view (three different positions across screen sizes seems one too many 😅) |
Thanks for the feedback, @delucis!
I like this!
The collapsable sections would be so nice, this would make (looks like I just found more sidebar-related stuff to PR, yay!) |
Here is the PR: #735 |
Done in #735. Thanks @Yan-Thomas! 🥳 |
Currently, we put the MoreMenu.astro at the bottom of the page on mobile views, which means that they only rarely get seen..
Two options would be:
The text was updated successfully, but these errors were encountered: