-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[ColorSystem][MessageIndicator] Improve contrast and use color system #2428
Conversation
💦 Potential splash zone of changes introduced to
DetailsAll files potentially affected (total: 3)📄
|
4c1e3c3
to
bdf9550
Compare
38874a7
to
3f7fada
Compare
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.
Couple small things, otherwise nice improvement!
c00a89f
to
93846b9
Compare
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.
👍
e686b73
to
c433d3a
Compare
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.
c433d3a
to
1c04a7c
Compare
1c04a7c
to
82f73c1
Compare
WHY are these changes introduced?
To improve the contrast of
MessageIndicator
shown whenmessage
is set onTopBar.UserMenu
. Thanks for the design @AdamWhitcroft 🎉cc @mirualves for UI Kit
WHAT is this pull request doing?
Added a border for the message indicator colored with the
--p-surface-background
CSS variable. The indicator's color now uses the--p-highlight-icon
CSS variable.How to 🎩
🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines
Copy-paste this code in
playground/Playground.tsx
:🎩 checklist
README.md
with documentation changes