fix(cxl-ui): add missing pointer on hover for dashboard elements

conversionxl / aybolit

Lightweight web components library built with LitElement.

https://conversionxl.github.io/aybolit/

MIT License

7 stars 8 forks source link

fix(cxl-ui): add missing pointer on hover for dashboard elements #326

Closed freudFlintstone closed 10 months ago

freudFlintstone commented 10 months ago

https://app.clickup.com/t/86ay8bcjn

[x] Featured course slider left/right arrows + hover state needed
[x] Accordion header when it's opened
[x] Cards "View lesson/course" link
[x] Accordion slider left/right arrows + hover state needed

github-actions[bot] commented 10 months ago

size-limit report 📦

Path	Size
packages/cxl-ui/pkg/dist-web/cxl-ui.js	61.59 KB (+0.15% 🔺)
packages/cxl-ui/pkg/dist-web/cxl-ui-jwplayer.js	11.87 KB (0%)
packages/cxl-ui/pkg/dist-web/cxl-ui-playbooks.js	27.11 KB (+0.14% 🔺)
packages/cxl-ui/pkg/dist-web/vendor.js	135.58 KB (0%)
packages/cxl-ui/pkg/dist-web/cxl-ui-jwplayer.js, packages/cxl-ui/pkg/dist-web/cxl-ui-playbooks.js, packages/cxl-ui/pkg/dist-web/cxl-ui.js, packages/cxl-ui/pkg/dist-web/manifest.js, packages/cxl-ui/pkg/dist-web/unresolved.js, packages/cxl-ui/pkg/dist-web/vendor.js	237.3 KB (+0.06% 🔺)

freudFlintstone commented 10 months ago

[ ] Cards "View lesson/course" link @pawelkmpt What is exactly the issue with this one? It has the pointer and hover state, although this particular vaadin theme (tertiary) makes the hover state change very subtle. Should I try to improve that?

pawelkmpt commented 10 months ago

[ ] Cards "View lesson/course" link @pawelkmpt What is exactly the issue with this one? It has the pointer and hover state, although this particular vaadin theme (tertiary) makes the hover state change very subtle. Should I try to improve that?

In Storybook, all stories from this list:

Screenshot 2023-09-12 at 15 10 41

Subtle hover is there but pointer is not