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

cursor:pointer on static paragraphs in navigation sidebar #69

Closed
patrickhlauke opened this issue Nov 21, 2019 · 1 comment · Fixed by #73
Closed

cursor:pointer on static paragraphs in navigation sidebar #69

patrickhlauke opened this issue Nov 21, 2019 · 1 comment · Fixed by #73

Comments

@patrickhlauke
Copy link
Contributor

similar to #54 and #66 the navigation sidebar has static content ("Documentation" and "Implementations") where the mouse cursor is forced to appear as if they were actually clickable/interactive via cursor:pointer. This is misleading/confusing.

hovering over 'Documentation' shows the 'hand' mouse cursor even though the text is static/not a link

@patrickhlauke
Copy link
Contributor Author

patrickhlauke commented Nov 21, 2019

this seems, at its core, to have the same root cause as #54 since in layout.scss the styles for .section-layoutcontainer>aside>nav>ul>li>p (and not just for ... ul>li>a) include @include header-style($fg-secondary), and in turn that mixin defines the cursor:pointer style

patrickhlauke added a commit to patrickhlauke/act-rules-web that referenced this issue Nov 21, 2019
While it's true that most elements that use this mixin are also interactive, it is also used on static elements ... and here, having the explicit `cursor:pointer` mouse cursor (the "hand" one) is misleading and confusing, as it makes those static elements appear to be interactive/clickable to sighted mouse users.

Closes act-rules#54
Closes act-rules#69
jeeyyy pushed a commit that referenced this issue Nov 21, 2019
While it's true that most elements that use this mixin are also interactive, it is also used on static elements ... and here, having the explicit `cursor:pointer` mouse cursor (the "hand" one) is misleading and confusing, as it makes those static elements appear to be interactive/clickable to sighted mouse users.

Closes #54
Closes #69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant