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

Collapsed view menu button should be on the same side as the menu #1312

Closed
BioTurboNick opened this issue May 4, 2020 · 6 comments · Fixed by #2076
Closed

Collapsed view menu button should be on the same side as the menu #1312

BioTurboNick opened this issue May 4, 2020 · 6 comments · Fixed by #2076
Labels
Format: HTML Related to the default HTML output help wanted Type: Enhancement

Comments

@BioTurboNick
Copy link

I don't know if this is standard, but I saw it used in docs.juliaplots.org so reporting it here, presuming this package generated it.

Annotation 2020-05-04 153736

Menu appears on the left but the menu button is on the top right. This just doesn't work well. Please keep them on the same side.

@mortenpi mortenpi added Format: HTML Related to the default HTML output Status: Speculative It's unknown if this is something that we wan't to do labels May 5, 2020
@mortenpi
Copy link
Member

mortenpi commented May 5, 2020

Yep, it comes from here. I'll leave this open for now if there are others who'd like to chime in, but personally I am inclined to leave it unchanged.

That button is aimed for mobile users, in which case it's not really that disconnected from the menu:

Screenshot_20200505-123454

I prefer having it there, since then I can just use my thumb to tap it, without either having to reach over the screen or using the other hand. Looking around on some websites, it's also quite common to have the menu button on the right, so it should be a familiar UX for the users.

@BioTurboNick
Copy link
Author

BioTurboNick commented May 5, 2020

I can understand that motivation.

Do you have an example of a right-side menu button where the menu doesn't open adjacent to the button? Because it wasn't familiar to this user. The menu is on the left side, so that's where I looked for a way to activate it.

It's also going into a collapsed view when the window is occupying a bit less than half of my 3840x2160 (150% dpi) display. So if it's a mobile feature, perhaps it's activating at too wide a view. Traversing that distance with the mouse every time is annoying.

If unchanged, it would at least be really helpful if there was some nub at the left edge to activate.

@xgdgsc
Copy link

xgdgsc commented May 15, 2020

It' s strange to serve the mobile users. For me I like tiling windows half of screen on 1080p screens. It just triggers the collapse which makes it very far from reach. I don' t find that button a familiar UX because it was near the github button to edit the doc, I thought it was something related to editing pages at first thus dare not try when I didn' t find it at left.

@yuyichao
Copy link

I hit this issue more than once last week. I also use left/right split screen on laptop. I know the table of contents is on the left and it always takes me a while to figure out how to access it.

@mortenpi mortenpi added Type: Enhancement help wanted and removed Status: Speculative It's unknown if this is something that we wan't to do labels Jun 3, 2020
@mortenpi
Copy link
Member

mortenpi commented Jun 3, 2020

Do you have an example of a right-side menu button where the menu doesn't open adjacent to the button?

I concede that having the hamburger on the left is more common, also e.g. in the Android ecosystem. The Guardian does have it on the right. I also see examples where it is positioned on the right, but the menu opens from the top, so it's not quite the same.

In any case, I do think now that it would be an improvement to have it on the left. So if someone wants to fiddle with the (S)CSS and provide a functioning mockup, I'd be happy to make the change.

For me I like tiling windows half of screen on 1080p screens. It just triggers the collapse which makes it very far from reach.

Separate from the hamburger issue, we should probably adjust when exactly the sidebar collapses, to better serve this use case (related: #1324).

@inkydragon
Copy link
Contributor

inkydragon commented Mar 18, 2023

if someone wants to fiddle with the (S)CSS and provide a functioning mockup, I'd be happy to make the change.

Is it enough to just move the hamburger button to the left?

I think we need to fix this before we can add the new TOC sidebar (#1688).
Since the button for TOC will be on the right side

image
ref: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/aside

# Hamburger on mobile
push!(navbar_right.nodes, a[
"#documenter-sidebar-button.docs-sidebar-button.docs-navbar-link.fa-solid.fa-bars.is-hidden-desktop",
:href => "#",
])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Format: HTML Related to the default HTML output help wanted Type: Enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants