-
-
Notifications
You must be signed in to change notification settings - Fork 198
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
toc: rework UX #167
Comments
Continuing from #151
Check out the test page https://process-sdk-draft.doc.moovapps.com/rest/. It is not a problem when the toc exceeds screen height. Make the browser window smaller until it covers the toc. Now the toc is scrollable. |
That's maybe the point here: I don't use a mouse but a trackpad. Therefore: no wheels attached ;-) |
I just tried on my laptop which has a trackpad and I have no problems scrolling the TOC frankly. It behaves just the same as when using the mouse wheel. EDIT: Does your trackpad not have a scroll functionality?? |
You forced me to RTFM ;-) So at least, I learned how to scroll with my touchpad, today. Yes, it scrolls but while scrolling there doesn't appear a small scroll bar as in your image. Really wild issue |
Hah! You have utterly amazed me by that comment! :D
Huh?! Weird. I actually noticed that it even appears on hover, so even if for some reason a user would not have scroll, he could still drag it. EDIT: Not quite true, it needs an "initial" scroll event for some reason... after that it can be hovered. UPDATE: |
The main usability concerns were addressed in #268. This is not the big rework of this issue but at the moment I have no plans for more drastic changes. |
The current TOC implementation isn't usable on touch devices as it's only working on MouseOver.
Reference: matcornic/hugo-theme-learn#67
The new TOC implementation should:
This was discussion started in #151 and is now keept track in its own issue.
Consider PR in matcornic/hugo-theme-learn#101.
The text was updated successfully, but these errors were encountered: