feat(MessageBar): Add support for high contrast #734
Conversation
|
Preview: https://chatbot-pr-chatbot-734.surge.sh A11y report: https://chatbot-pr-chatbot-734-a11y.surge.sh |
rebeccaalpert
force-pushed
the
message-bar-hcc
branch
from
6ff5b87 to
0638064
Compare
rebeccaalpert
force-pushed
the
message-bar-hcc
branch
from
0638064 to
340b4a0
Compare
|
Added tokens per feedback; let me know if you want other changes here! |
|
Code looks good - just want to check with @lboehling that the border shouldn't increase to 2 when hovering in HC? |
There was a problem hiding this comment.
LGTM, likewise to above would want to know if the MessageBar looks fine @lboehling when the hasAiIndicator prop is passed in (the one that gives the MessageBar the nifty gradient border)
lboehling
left a comment
lboehling
left a comment
There was a problem hiding this comment.
good call @srambach, the border should increase to 2px on hover, 3px on click. Using the border--width--control--hover and border--width--control--clicked tokens for those states should give the appropriate border widths when high contrast is enabled.
|
🎉 This PR is included in version 6.5.0-prerelease.17 🎉 The release is available on: Your semantic-release bot 📦🚀 |
5 participants
Demo: https://chatbot-pr-chatbot-734.surge.sh/patternfly-ai/chatbot/ui#message-bar-with-speech-recognition-and-file-attachment