Skip to content

Conversation

@nqst
Copy link
Contributor

@nqst nqst commented Dec 24, 2025

Changes:

  • Updated the title to use the singular form and capitalization (Notification Settings instead of Notifications settings).
  • Set the same top margin for the Notification Settings panel group as in Account Settings.
  • Normalized the corner radius in .notification__status so it looks more balanced with the parent .panel, especially on mobile.
  • Changed the Chrome label to Google Chrome in the "Help me fix this" instructions for macOS. This matches how it appears in System Settings and may save time when searching the list.
  • Added a missing comma.

Preview

Before After
push-before push-after
chrome-before chrome-after
Screenshot 2025-12-24 at 16 55 36@2x

P.S. Merry Christmas everyone! 🎄

@jorgemanrubia jorgemanrubia requested a review from andyra January 7, 2026 15:09
@jorgemanrubia
Copy link
Member

This looks good to me, what do you think @andyra?

@andyra
Copy link
Contributor

andyra commented Jan 7, 2026

👏 Lovely work, @nqst! Thanks for the improvements.

Aside: I wonder if there's a clever way to get corner radiuses to pair well with the .panel component—as long as we know the panel radius and padding value, it would be easy enough to calculate like so:

--inner-radius: calc(var(--panel-radius) - var(--panel-padding));

@andyra andyra merged commit f58bb33 into basecamp:main Jan 7, 2026
5 checks passed
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.

3 participants