Skip to content

Conversation

aaronjpark
Copy link

@aaronjpark aaronjpark commented Mar 19, 2025

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:

Screenshot 2025-03-19 at 12 21 12 AM

after: changed the size prop to small to make the hitbox area bigger:

Screenshot 2025-03-19 at 12 21 49 AM

[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:
Screenshot 2025-03-25 at 8 05 46 PM

after:
Screenshot 2025-03-25 at 8 36 44 PM


This change is Reviewable

@aaronjpark aaronjpark changed the title chore: Add a bigger "hitbox" to calendar sidebar buttons #519 chore: Add a bigger "hitbox" to calendar sidebar buttons Mar 19, 2025
@aaronjpark aaronjpark changed the title chore: Add a bigger "hitbox" to calendar sidebar buttons style: Add a bigger "hitbox" to calendar sidebar buttons Mar 19, 2025
@DereC4 DereC4 self-requested a review April 6, 2025 20:35
Copy link
Member

@DereC4 DereC4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@doprz doprz requested a review from IsaDavRod April 15, 2025 19:11
@Razboy20
Copy link
Member

Are the screenshots up to date? If so, the spacing should not be affected

@DereC4
Copy link
Member

DereC4 commented May 31, 2025

why has this still not been merged bruh i thought we got over it eons ago

Copy link
Member

@Razboy20 Razboy20 left a 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/

@Razboy20 Razboy20 changed the title style: Add a bigger "hitbox" to calendar sidebar buttons style: UTRP-14: Add a bigger "hitbox" to calendar sidebar buttons Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a bigger "hitbox" to calendar sidebar buttons by changing their variant to Small

5 participants