Skip to content

Refresh: Standardize border widths at 2px (Fix #15428)#15435

Merged
craigcook merged 1 commit into
mainfrom
m24-borders
Nov 4, 2024
Merged

Refresh: Standardize border widths at 2px (Fix #15428)#15435
craigcook merged 1 commit into
mainfrom
m24-borders

Conversation

@stephaniehobson
Copy link
Copy Markdown
Contributor

One-line summary

Standardize border widths at 2px for refresh.

Significant changes and points to review

Tess is standardizing on 2px for all borders and dividers. I added the variable $border-width and changed border and border-like properties to use it.

Issue / Bugzilla link

Fix #15428

Testing

🤷‍♀️ No CSS errors? Things still have borders?

@stephaniehobson stephaniehobson added Needs Review Awaiting code review Review: µ Code review time: 5 minutes or less Frontend HTML, CSS, JS... client side stuff Refresh Work related to the 2024 site refresh labels Nov 1, 2024
Copy link
Copy Markdown
Contributor

@craigcook craigcook left a comment

Choose a reason for hiding this comment

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

🐗

@craigcook craigcook merged commit 23da641 into main Nov 4, 2024
@craigcook craigcook deleted the m24-borders branch November 4, 2024 19:52
@stephaniehobson stephaniehobson removed the Needs Review Awaiting code review label Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend HTML, CSS, JS... client side stuff Refresh Work related to the 2024 site refresh Review: µ Code review time: 5 minutes or less

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refresh: Standardize border width at 2px

2 participants