Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use Icon in almost all the places #2036

Merged
merged 6 commits into from
Jan 25, 2018
Merged

use Icon in almost all the places #2036

merged 6 commits into from
Jan 25, 2018

Conversation

giladgray
Copy link
Contributor

Related to #2028

Use Icon in almost all the places. The final outstanding usage is in TimePicker, and I have a separate PR for that as it's a larger refactor.

This change is unrelated to introducing SVG icons and will simply enable these components to pick up that change for free.

@giladgray giladgray changed the title Gg/more icons use Icon in almost all the places Jan 24, 2018
@blueprint-bot
Copy link

measureTextWidth returns number

Preview: documentation | landing | table

@adidahiya
Copy link
Contributor

pls fix conflicts

@blueprint-bot
Copy link

Merge branch 'develop' of github.com:palantir/blueprint into gg/more-icons

Preview: documentation | landing | table

@adidahiya
Copy link
Contributor

more conflicts

@blueprint-bot
Copy link

Merge branch 'develop' of github.com:palantir/blueprint into gg/more-icons

Preview: documentation | landing | table

@adidahiya adidahiya merged commit 33e5d96 into develop Jan 25, 2018
@adidahiya adidahiya deleted the gg/more-icons branch January 25, 2018 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants