Differential D15157
[Cashtab] Better button sizing, spacing, and hover Authored by bytesofman on Jan 12 2024, 23:14.
Details
T2650 Make button take up all alotted space, so you do not have to click just on the icon. Add hover effect for non-mobile devices. Ref https://stackoverflow.com/questions/17233804/how-to-prevent-sticky-hover-effects-for-buttons-on-touch-devices for why we need the media query (otherwise if you push a button, it gets the hover state on mobile) Can verify mobile at https://cashtab-local-dev.netlify.app/ npm test, npm start and test hover effect
Diff Detail
Event TimelineComment Actions clickable button surface before  after  also now with hover effect for desktop  Note : the white borders in the above screenshot are to show element size, they are not implemented in this diff Comment Actions How about having the submenu (hamburger menu) rows also be magenta on hover like the buttons are? Right now they're the same blue that they are when they're the active page. Comment Actions Build Bitcoin ABC Diffs / Diff Testing (preview-cashtab) passed. Comment Actions The submenu is a bit offset so probably want to look into that (either in this diff or another one). Comment Actions FYI this is now broken for Cashtab because of the headers, Cashtab won't do anything not over https |