Skip to content

Use theme.semanticColors for error text and border of SendBox#1861

Merged
mgamis-msft merged 8 commits intomainfrom
mgamis/fix-sendbox-error-colors
May 11, 2022
Merged

Use theme.semanticColors for error text and border of SendBox#1861
mgamis-msft merged 8 commits intomainfrom
mgamis/fix-sendbox-error-colors

Conversation

@mgamis-msft
Copy link
Copy Markdown
Contributor

@mgamis-msft mgamis-msft commented May 5, 2022

What

  • Added semanticColors.errorText to light and dark themes to style error border of the SendBox component and the error text of the InputBoxComponent component

Why

https://skype.visualstudio.com/SPOOL/_workitems/edit/2779691

How Tested

Testing Sendbox border and InputBoxComponent text when there is a system error in storybook:
https://user-images.githubusercontent.com/79475487/166894179-3246e245-bc44-4e4e-98d2-590fff29abc0.mp4

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2022

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2022

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2022

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "ui change" label to the PR for updating the snapshot.

@mgamis-msft mgamis-msft changed the title Use theme.semanticColors for error text, borders, and background Use theme.semanticColors for error text and border of SendBox May 5, 2022
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2022

@JamesBurnside JamesBurnside mentioned this pull request May 10, 2022
3 tasks
@github-actions
Copy link
Copy Markdown
Contributor

@mgamis-msft mgamis-msft enabled auto-merge (squash) May 11, 2022 21:29
@github-actions
Copy link
Copy Markdown
Contributor

@mgamis-msft mgamis-msft merged commit bdde3bc into main May 11, 2022
@mgamis-msft mgamis-msft deleted the mgamis/fix-sendbox-error-colors branch May 11, 2022 21:51
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.

4 participants