-
Notifications
You must be signed in to change notification settings - Fork 912
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
Add cursor: pointer to details summary on /cookie-settings/ page (Fixes #14781) #14782
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #14782 +/- ##
=======================================
Coverage 77.18% 77.18%
=======================================
Files 159 159
Lines 8239 8239
=======================================
Hits 6359 6359
Misses 1880 1880 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I welcome this good UX change, r+! I left a tiny comment that I'll leave up to you to update or not 🎀
@@ -80,6 +80,11 @@ main { | |||
|
|||
.cookie-summary { | |||
border-bottom: 1px solid $color-light-gray-40; | |||
|
|||
// Temporary fix until https://github.com/mozilla/protocol/issues/948 is added | |||
&:hover { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a blocker, but I don't think &:hover
is really necessary since it performs the same action
64b26f3
to
d7ffa89
Compare
One-line summary
Adds
cursor: pointer;
to<summary>
elements on hover in /cookie-settings/ pageIssue / Bugzilla link
#14781
Testing
http://localhost:8000/en-US/privacy/websites/cookie-settings/