-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Layout issues in documentation (horizontal navbar) #12558
Comments
The line breaks in the navbar only occur for me when the browser window is too narrow. But yes, I also think that our website is currently not very nice for the following reasons:
Personally, I'd switch to Material for MkDocs, because it is visually just so much better. But I guess this is too big of a change that it won't be an option (and I don't know how easy it is to import existing Sphinx stuff). |
From an a11y perspective, they also don't honor "reduce motion" settings of my OS/browser (in that case, the zooming motion should be disabled).
Not an option because we're stuck with Sphinx … 😬😵 Main advantage of Material for MkDocs for me is the search-as-you-type feature, it's such an essential thing to have these days and it's embarrassing our theme doesn't support this yet. Our search is horrendous if you don't already know what you're looking for. Progress on this feature in pydata-sphinx-theme seems to be stalled. – But that's another issue really. Let's fix the navbar for now! :) |
Right, please just ignore my remark about MkDocs, this is just wishful thinking. All other points (sidebar, navbar, organization) still stand though. |
This should now be fixed, the overnight builds just hadn't completed yet For all the UI/UX stuff yes it would be great to improve any problems. PRs welcome to improve things. FWIW I can replicate the top bar stuff once I shrink my window on Linux so that part at least seems to have to do with window size and when things break (so maybe removing some social icons would already help) vs |
Unfortunately this doesn't seem to fix it for me on Safari on macOS: tried different screen sizes and window widths, always getting the same result |
I can confirm that Safari on macOS always renders the navbar in two rows. Edit: Same on Chrome on macOS – maybe it's a macOS thing? |
Re the version chooser, for me it still shows "Choose version" instead of the actual version number. |
Same, I tried the devel docs |
https://pydata-sphinx-theme.readthedocs.io/en/latest/user_guide/layout.html#overview-of-theme-layout
I think this is because our "last updated" JS uses
Not sure what this refers to?
This is not a bug, and not about the theme. Open a separate issue if you want to discuss MNE's choices about website content.
Did you try the workaround in that thread? pydata/pydata-sphinx-theme#1238 (comment) If it works, PR welcome.
This is not a bug, and not about the theme. Open a separate issue if you want to discuss MNE's choices about website content.
This is not a bug, and not about the theme. Open a separate issue if you want to discuss MNE's choices about website content.
This is not a bug, and not about the theme. Open a separate issue if you want to discuss MNE's choices about website content.
They do have text when you aren't hovering though... Do you mean there's no "alt" text? That said, I agree with @hoechenberger that they should respect the "minimal motion" browser setting.
Can you link to the previous discussion / reopen it there? These omnibus issues make it really hard to keep track of everything. (Also: this is not a bug, and not about the theme).
For context on why it is purple, I invite you to read through:
As I said in #12565 (comment) I'm open to customizing the colors we use, but not casually and not at the expense of a11y guideline compliance. |
Once I hover over them, I don't know what this UI element was supposed to resemble anymore, as the text is gone and the images could mean anything. The text should never disappear. My proposal would be to have the text below or above each of those tiles, and remain always visible, hovered or not. |
@hoechenberger respectfully: Words like "embarrassing" and "horrendous" are not helpful here. I know that you know how OSS works, and that features like this will only ever get implemented if someone finds (almost certainly volunteers) the time to make it happen. As a maintainer of the theme I can tell you with certainty: there are only a few of us, we all have other higher-priority responsibilities, and the backlog of bugs and feature requests is a huge source of stress. Bitter complaints like this do not help. |
I don't question that. I do question our use of this theme when there's such obvious usability problems and lack of resources to fix them, though. |
OK, so
and
are the same issue / should have the same solution I offered above ( |
That could be, I honestly don't remember exactly what I meant :( Sorry about the confusion!!! your proposed fix seems like a good idea! |
Fair enough. Please do so politely (and in a separate issue) if you want to have a serious discussion about dropping Sphinx / Pydata Sphinx Theme. |
Just FYI I am on a 13 inch MacBook, and the topbar line breaks occur even when setting the safari browser to full screen. So for folks like me with smaller MacBooks, the browser window will always be too narrow. |
@scott-huberty did you try #12571? |
No, sorry, wasn't sure if the topbar line breaks were being addressed there! |
Thanks For the tip @hoechenberger - Looks like I'm just using the Default resolution, though I tried all the resolution options listed below, and none of them changed the topbar line break behavior : ( ![]() |
Can you hover over that "Default" tile and let us know which resolution is displayed? |
Yes, 1440 x 990 And if needed, my available resolutions (left to right) are: [ |
thanks! |
The navbar issue has been resolved by #12571. The only thing remaining is the version chooser. Just to be clear, the versions are correctly shown in the drop-down menu, but the issue here is that the title of the chooser should include the currently selected version (e.g. "1.7.0") as opposed to "Choose version" (please correct me if I'm wrong @hoechenberger). |
https://mne.tools/stable/index.html
This is with Safari on macOS.
The text was updated successfully, but these errors were encountered: