-
Notifications
You must be signed in to change notification settings - Fork 78
style: UTRP-14: Add a bigger "hitbox" to calendar sidebar buttons #563
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
base: main
Are you sure you want to change the base?
style: UTRP-14: Add a bigger "hitbox" to calendar sidebar buttons #563
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are the screenshots up to date? If so, the spacing should not be affected |
why has this still not been merged bruh i thought we got over it eons ago |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's refactor this to not adjust the actual spacing and placement of the icons, but use ::after with inset.
Take a look at this article: https://51bits.com/expanded-hit-areas/
This PR "potentially" closes #519
[Task 1 - Make the HitBox Area of Sidebar and Plus Buttons Bigger]
DISCLAIMER: the screenshot only shows the sidebar button. All 3 buttons (sidebar, +, settings) have been changed.
Before: the size of the hitbox for most of the sidebar buttons were extremely small:
after: changed the size prop to small to make the hitbox area bigger:
[Task 2 - Make the HitBox Area of Social Links Bigger]
DISCLAIMER: #519 doesn't specify how big the hitbox for the social links should be. Talked to Derek in-person and Derek said p-2 looks good. The Red Borders were added for visual reference.
before:

after:

This change is