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

fix(Icon): update hand icon in Teams theme #1782

Merged
merged 14 commits into from
Aug 19, 2019
Merged

fix(Icon): update hand icon in Teams theme #1782

merged 14 commits into from
Aug 19, 2019

Conversation

lucivpav
Copy link
Contributor

@lucivpav lucivpav commented Aug 9, 2019

This PR fixes the hand icon in Teams theme. Fixes #1766.

Before
image

After
image

@vercel vercel bot temporarily deployed to staging August 9, 2019 14:00 Inactive
@codecov
Copy link

codecov bot commented Aug 9, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1782   +/-   ##
=======================================
  Coverage   69.76%   69.76%           
=======================================
  Files         875      875           
  Lines        7505     7505           
  Branches     2209     2182   -27     
=======================================
  Hits         5236     5236           
  Misses       2261     2261           
  Partials        8        8
Impacted Files Coverage Δ
...rc/themes/teams/components/Icon/svg/icons/hand.tsx 100% <ø> (ø) ⬆️
.../components/Icon/svg/ProcessedIcons/icons-hand.tsx 100% <ø> (ø) ⬆️

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 ec663a4...aeb9199. Read the comment docs.

@miroslavstastny
Copy link
Member

@codepretty - could you please review this one?

@vercel vercel bot temporarily deployed to staging August 15, 2019 08:18 Inactive
@layershifter layershifter changed the title fix(teams): hand icon fix(Icon): update hand icon in Teams theme Aug 16, 2019
@layershifter layershifter added the 🧰 fix Introduces fix for broken behavior. label Aug 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🧰 fix Introduces fix for broken behavior. 🚀 ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hand icon is slightly cut off at the top
4 participants