UI: Fix incorrect cursor style on Filter options and RESET button (#436) #447
Conversation
vbakke
left a comment
vbakke
left a comment
There was a problem hiding this comment.
Thank you for pointing this out, @Biswas-Samrat.
Can you add the pointer hand to the filter chips in the Overview page as well? So that all pages stays consistent?
Also, when we are tiding CSS, can you base on the defined CSS variables (such as --primary-color)? That way we slowly get rid of all the hard coded color codes all over the place :)
|
Hi @vbakke, I have updated the PR to include cursor: pointer for the filter chips on the Overview page. |
vbakke
left a comment
vbakke
left a comment
There was a problem hiding this comment.
Thank you @Biswas-Samrat. Please also update the colour, and sign the submissions. :)
| } | ||
|
|
||
| .reset-button:hover { | ||
| background-color: #4caf50; |
There was a problem hiding this comment.
We have a number of "green colours" scattered around the CSS, but let's try to reduce the numbers. Please base the colours on the var()s defined in custom-theme.scss, whenever possible.
2 participants
Fixes a UI issue where interactive filter elements and the RESET button were using the default cursor. They now correctly display the pointer cursor on hover.
Closes #436