readthedocs / readthedocs/ext-theme
Notifications: improve styling of close icon
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 12
- Forks
- 6
- Avg merge
- 13h 25m
- Merged PRs (30d)
- 1
Description
I think many users are missing the close button icon in the notification, it's really common to jump into impersonate and see a dismissable notification lingering around. The close icon is fairly easy to miss. This is just a guess though, I haven't had any firm advice from users on this.
This might be able to be improved at the template level but it might need to be fixed at the CSS level as .close.icon is special inside a .ui.message.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the notification template and the CSS behavior of .close.icon inside .ui.message, as mentioned in the issue. Confirm the close icon is visually easy to miss, then verify that the updated styling makes the dismiss control clearly visible in the impersonation notification.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- css, html
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100