Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

fix(Chat Message): update at mention colors#1232

Merged
codepretty merged 8 commits intomasterfrom
fix/at-mention-color
Apr 20, 2019
Merged

fix(Chat Message): update at mention colors#1232
codepretty merged 8 commits intomasterfrom
fix/at-mention-color

Conversation

@codepretty
Copy link
Collaborator

@codepretty codepretty commented Apr 17, 2019

The at mention color for the bar should be different from the at mention nubbin color. The nubbin and bar colors for high contrast should all be yellow.

Teams default
image

Teams dark
image

Teams contrast
image

@codecov
Copy link

codecov bot commented Apr 17, 2019

Codecov Report

Merging #1232 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1232   +/-   ##
=======================================
  Coverage   71.57%   71.57%           
=======================================
  Files         730      730           
  Lines        5579     5579           
  Branches     1634     1612   -22     
=======================================
  Hits         3993     3993           
  Misses       1581     1581           
  Partials        5        5
Impacted Files Coverage Δ
...h-contrast/components/Chat/chatMessageVariables.ts 0% <ø> (ø) ⬆️
...emes/teams/components/Chat/chatMessageVariables.ts 0% <ø> (ø) ⬆️
...teams-dark/components/Chat/chatMessageVariables.ts 0% <ø> (ø) ⬆️
.../themes/teams/components/Chat/chatMessageStyles.ts 2.7% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3bb8de7...a6aeba1. Read the comment docs.

CHANGELOG.md Outdated
- Fix overflowing focus outline for `Grid` items for Teams theme @Bugaa92 ([#1195](https://github.com/stardust-ui/react/pull/1195))
- Fix routing for accessibility documentation @sophieH29 ([#1208](https://github.com/stardust-ui/react/pull/1208))
- Fix `content` prop type in `Dialog` @layershifter ([#1212](https://github.com/stardust-ui/react/pull/1212))
- Fix themeing for at mention nubbin and left bar in `ChatMessage` @codepretty ([#1232](https://github.com/stardust-ui/react/pull/1232))
Copy link
Contributor

Choose a reason for hiding this comment

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

Add in the changelog entry in Teams theme

@codepretty codepretty merged commit ac07054 into master Apr 20, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix/at-mention-color branch April 20, 2019 18:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants