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

fix(theme): don't let navbar obstruct clicks to top part of scrollbar #1168

Merged
merged 1 commit into from Aug 15, 2022
Merged

fix(theme): don't let navbar obstruct clicks to top part of scrollbar #1168

merged 1 commit into from Aug 15, 2022

Conversation

curtgrimes
Copy link
Contributor

On desktop screen sizes, if the sidebar has a scrollbar, the very top
part of the scrollbar can't be clicked on because the navbar obstructs
it due to having a higher z-index. (This higher z-index is necessary so
that the navbar is visible.) With this fix, the navbar container has
pointer-events: none; so that clicks to the scrollbar underneath will
work. Child elements of the navbar that need to receive clicks have
pointer-events: all; turned on for them.

fix vueuse/vueuse#2086

Before

The red highlighted part of the scrollbar could not be clicked and dragged:

image

After

The part of the scrollbar above can now be clicked and dragged, and all of the other parts of the sidebar and navbar remain clickable as before.

On desktop screen sizes, if the sidebar has a scrollbar, the very top
part of the scrollbar can't be clicked on because the navbar obstructs
it due to having a higher z-index. (This higher z-index is necessary so
that the navbar is visible.) With this fix, the navbar container has
`pointer-events: none;` so that clicks to the scrollbar underneath will
work. Child elements of the navbar that need to receive clicks have
`pointer-events: all;` turned on for them.

fix vueuse/vueuse#2086
@brc-dd brc-dd merged commit 44953dc into vuejs:main Aug 15, 2022
@brc-dd
Copy link
Member

brc-dd commented Aug 15, 2022

Thanks!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Left scroll bar not working in top
2 participants