Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Make the cursor a pointer for summaries #8597

Closed
wants to merge 0 commits into from

Conversation

wooster0
Copy link
Contributor

@wooster0 wooster0 commented May 14, 2022

I thought it might look nicer to make the cursor a pointer if the user hovers over summaries. I think it can help indicate to the user that this text is actually clickable, just like buttons.
Here's one example where I expect this to be effective (the "Advanced"):
image
I thought of also adding user-select: none; but buttons don't use it either so I didn't use it here.

Signed-off-by: r00ster91 <r00ster91@proton.me>

Here's what your changelog entry will look like:

✨ Features

  • Make the cursor a pointer for summaries (#8597). Contributed by @r00ster91.

@wooster0 wooster0 requested a review from a team as a code owner May 14, 2022 13:47
@t3chguy t3chguy added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label May 14, 2022
Copy link
Contributor

@SimonBrandner SimonBrandner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM codewise; this might need a design review - let me check

@robintown robintown requested a review from a team May 14, 2022 15:07
@amshakal amshakal requested review from amshakal and removed request for a team May 31, 2022 16:43
@MadLittleMods MadLittleMods added the Z-Community-PR Issue is solved by a community member's PR label Jun 1, 2022
@wooster0 wooster0 closed this Oct 14, 2022
@wooster0 wooster0 temporarily deployed to Netlify October 14, 2022 10:49 Inactive
@wooster0 wooster0 temporarily deployed to Cypress October 14, 2022 10:50 Inactive
@kerryarchibald kerryarchibald temporarily deployed to Netlify October 14, 2022 10:54 Inactive
@kerryarchibald kerryarchibald temporarily deployed to Cypress October 14, 2022 10:55 Inactive
@kerryarchibald kerryarchibald temporarily deployed to Cypress October 14, 2022 10:55 Inactive
@hughns hughns temporarily deployed to Netlify October 14, 2022 11:01 Inactive
@hughns hughns temporarily deployed to Cypress October 14, 2022 11:01 Inactive
@hughns hughns temporarily deployed to Cypress October 14, 2022 11:01 Inactive
@hughns hughns temporarily deployed to Cypress October 14, 2022 11:01 Inactive
@justjanne justjanne temporarily deployed to Netlify October 14, 2022 11:10 Inactive
@kerryarchibald kerryarchibald temporarily deployed to Netlify October 14, 2022 11:39 Inactive
@kerryarchibald kerryarchibald temporarily deployed to Cypress October 14, 2022 11:39 Inactive
@kerryarchibald kerryarchibald temporarily deployed to Cypress October 14, 2022 11:40 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants