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

feat(Icon): export play and pause icons in Teams theme #1189

Merged
merged 6 commits into from
Apr 9, 2019

Conversation

layershifter
Copy link
Member

@layershifter layershifter commented Apr 8, 2019

Fixes #1187.

This PR add new play and pause icons, FontAwesome uses exactly these names.

image

@codecov
Copy link

codecov bot commented Apr 8, 2019

Codecov Report

Merging #1189 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1189      +/-   ##
==========================================
+ Coverage   82.44%   82.46%   +0.02%     
==========================================
  Files         738      740       +2     
  Lines        8743     8753      +10     
  Branches     1171     1237      +66     
==========================================
+ Hits         7208     7218      +10     
  Misses       1519     1519              
  Partials       16       16
Impacted Files Coverage Δ
...nents/Icon/svg/ProcessedIcons/icons-play-pause.tsx 100% <ø> (ø) ⬆️
.../components/Icon/svg/ProcessedIcons/icons-play.tsx 100% <ø> (ø) ⬆️
...c/themes/teams/components/Icon/svg/icons/pause.tsx 100% <100%> (ø)
...rc/themes/teams/components/Icon/svg/icons/play.tsx 100% <100%> (ø)
...rc/themes/teams/components/Icon/svg/icons/index.ts 100% <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 16b9fd0...a5faca2. Read the comment docs.

CHANGELOG.md Outdated
@@ -27,6 +27,9 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
- Fix `rotate` prop on `Icon` not working in `rtl` @mnajdova ([#1179](https://github.com/stardust-ui/react/pull/1179))
- `FocusTrapZone` - Do not propagate any keyboard events @sophieH29 ([#1180](https://github.com/stardust-ui/react/pull/1180))

### Features
- Export `flag` icon in Teams theme @layershifter ([#1189](https://github.com/stardust-ui/react/pull/1189))
Copy link
Contributor

Choose a reason for hiding this comment

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

a bit lost here - seems that PR description doesn't correspond to the actual changes introduced. Also, it seems that there is a mistaken item referenced in the PR description as 'fixed' - or it was about something specific from this referenced PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, copy paste is evil sometimes, updated 👍

@kuzhelov kuzhelov added the needs author feedback Author's opinion is asked label Apr 8, 2019
@layershifter layershifter removed the needs author feedback Author's opinion is asked label Apr 8, 2019
@layershifter layershifter merged commit 0ba5c0d into master Apr 9, 2019
@delete-merged-branch delete-merged-branch bot deleted the feat/export-play-pause-icons branch April 9, 2019 07:41
@layershifter layershifter mentioned this pull request Apr 9, 2019
11 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants