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

feat(Icon): add 'window-maximize' 'window-minimize' and 'window-restore' icons #1803

Merged
merged 5 commits into from
Aug 16, 2019

Conversation

codepretty
Copy link
Collaborator

@codepretty codepretty commented Aug 16, 2019

Added 3 new icons to Teams theme.

window-maximize window-minimize and window-restore

image

@codecov
Copy link

codecov bot commented Aug 16, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@d088785). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1803   +/-   ##
=========================================
  Coverage          ?   69.79%           
=========================================
  Files             ?      876           
  Lines             ?     7506           
  Branches          ?     2208           
=========================================
  Hits              ?     5239           
  Misses            ?     2259           
  Partials          ?        8
Impacted Files Coverage Δ
.../Icon/svg/ProcessedIcons/icons-chrome-maximize.tsx 100% <ø> (ø)
...con/svg/ProcessedIcons/icons-chrome-unmaximize.tsx 100% <ø> (ø)
.../Icon/svg/ProcessedIcons/icons-chrome-minimize.tsx 100% <ø> (ø)
...teams/components/Icon/svg/icons/windowMinimize.tsx 100% <100%> (ø)
...teams/components/Icon/svg/icons/windowMaximize.tsx 100% <100%> (ø)
.../teams/components/Icon/svg/icons/windowRestore.tsx 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 d088785...cb98199. Read the comment docs.

CHANGELOG.md Outdated
@@ -31,6 +31,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
- Add possibility for a Toolbar to rearrange its items according to space available @miroslavstastny ([#1657](https://github.com/stardust-ui/react/pull/1657))
- Edit `buttonBehavior` Adding aria-disabled when button has loading state @kolaps33 ([#1789](https://github.com/stardust-ui/react/pull/1789))
- Added `audio-off` and `clipboard-copied-to` icon to Teams theme @bcalvery ([#1792](https://github.com/stardust-ui/react/pull/1792))
- Added `window-maximize`, `window-minimize` and `window-unmaximize` icons to Teams theme @codepretty ([#1803](https://github.com/stardust-ui/react/pull/1803))
Copy link
Member

Choose a reason for hiding this comment

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

What about window-restore? As in FontAwesome:

image

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good find! I didn't notice those, so wasn't sure what to use for "unmaximize". Will update that to window-restore

@layershifter layershifter added ⚙️ enhancement New feature or request needs author feedback Author's opinion is asked and removed 🚀 ready for review labels Aug 16, 2019
@codepretty codepretty changed the title feat(Icon): add 'window-maximize' 'window-minimize' and 'window-unmaximize' icons feat(Icon): add 'window-maximize' 'window-minimize' and 'window-restore' icons Aug 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
⚙️ enhancement New feature or request needs author feedback Author's opinion is asked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants