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

Settings Icon on Now Playing Widget Cursor Should be Pointer #29

Closed
cdrani opened this issue Jul 28, 2023 · 0 comments · Fixed by #35
Closed

Settings Icon on Now Playing Widget Cursor Should be Pointer #29

cdrani opened this issue Jul 28, 2023 · 0 comments · Fixed by #35
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@cdrani
Copy link
Owner

cdrani commented Jul 28, 2023

Describe the bug

Hovering over or clicking on the Chorus/Settings Icon in the Now Playing Widget displays the cursor as the default. It should be a pointer to signify that it's clickable.

ScreenShots

image

CURRENT

On hover, the default cursor is used.

EXPECTED

Update to use pointer cursor.

IMPLEMENTATION
Apply cursor style attribute to #chorus-icon id:

chorus/styles.css

Lines 30 to 36 in ef998b9

#chorus-icon {
display: flex;
flex-wrap: wrap;
justify-content: center;
width: 2rem;
height: 2rem;
}

@cdrani cdrani added bug Something isn't working help wanted Extra attention is needed good first issue Good for newcomers labels Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
Status: Released
Development

Successfully merging a pull request may close this issue.

1 participant