-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix: settings icon margin in notification #3604
fix: settings icon margin in notification #3604
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@foolo it should be margin 🙇🏼 there is no space RN right to the button 🙇🏼 |
…a-margin-on-right-side
@bigint Done, I updated the "After" picture. Keeping the button itself narrow as before. |
@bigint Ah, now I think I understand what you mean. On desktop the cogwheel should have no margin. I updated the PR again. Sorry for lots of back and forth for this tiny PR :) |
What does this PR do?
🤖 Generated by Copilot at a25fa3a
Increased button padding in
Notification/Settings.tsx
component. This is a UI improvement for the web app.Before:
After:
Related issues
Fixes #3583
Type of change
Explanation of the changes
🤖 Generated by Copilot at a25fa3a
Settings.tsx
(link) to improve UI consistencyEmoji
🖥️📏🛠️