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

fix(Icon): Color & docs update for Teams theme #384

Merged
merged 8 commits into from
Oct 24, 2018

Conversation

codepretty
Copy link
Collaborator

@codepretty codepretty commented Oct 22, 2018

Updating the color for icon Teams theme.. An icon will inherit the current color of the font unless a new color is explicitly passed to icon.

Added new Color section to the doc site under Variations
image

@codecov
Copy link

codecov bot commented Oct 22, 2018

Codecov Report

Merging #384 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #384   +/-   ##
=======================================
  Coverage   91.72%   91.72%           
=======================================
  Files          41       41           
  Lines        1341     1341           
  Branches      172      172           
=======================================
  Hits         1230     1230           
  Misses        107      107           
  Partials        4        4

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 62dbe10...073e00d. Read the comment docs.

@codepretty codepretty changed the title Icon color & docs update for Teams theme color only fix(Icon): color & docs update for Teams theme only Oct 22, 2018
@codepretty codepretty changed the title fix(Icon): color & docs update for Teams theme only fix(Icon): Color & docs update for Teams theme Oct 22, 2018
@codepretty codepretty added redlines Update of the redlines for the mentioned component 🚀 ready for review labels Oct 22, 2018
Copy link
Contributor

@kuzhelov kuzhelov left a comment

Choose a reason for hiding this comment

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

please, see comments. Thanks!

@kuzhelov kuzhelov added needs author feedback Author's opinion is asked and removed 🚀 ready for review labels Oct 23, 2018
@codepretty codepretty added 🚀 ready for review and removed needs author feedback Author's opinion is asked labels Oct 23, 2018
Copy link
Contributor

@kuzhelov kuzhelov left a comment

Choose a reason for hiding this comment

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

lets, please, agree on the example provided before merging these changes - other than that am totally for merging it 👍

@kuzhelov kuzhelov added needs author feedback Author's opinion is asked and removed 🚀 ready for review labels Oct 23, 2018
@codepretty codepretty removed the needs author feedback Author's opinion is asked label Oct 23, 2018
Copy link
Contributor

@kuzhelov kuzhelov left a comment

Choose a reason for hiding this comment

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

please, update the CHANGELOG before merge - thanks!

@codepretty codepretty merged commit 03c9671 into master Oct 24, 2018
@codepretty codepretty deleted the feat/icon-color-redlines branch October 24, 2018 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready for merge redlines Update of the redlines for the mentioned component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants