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

[icons] feat: new icon "rocket" #4840

Merged
merged 5 commits into from
Aug 16, 2021
Merged

Conversation

millasml
Copy link
Contributor

@millasml millasml commented Aug 5, 2021

Fixes #0000

Checklist

  • Includes tests
  • Update documentation

Changes proposed in this pull request:

Added rocket and rocket-slant icons

Reviewers should focus on:

Making sure everything renders okay and that the right files were changed.

Screenshot

image

@palantirtech
Copy link
Member

Thanks for your interest in palantir/blueprint, @millasml! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@millasml millasml changed the title feat: new icon - rocket [icons] feat: new icon - rocket Aug 5, 2021
Copy link
Contributor

@adidahiya adidahiya left a comment

Choose a reason for hiding this comment

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

there's something wrong with the SVG syntax in the 16px rocket-slant, 16px rocket, and 20px rocket files. note how they use a <g clip-path> while the simpler 20px rocket-slant icon does not (that's the only one that renders correctly):

image

please make sure to pull from this branch to get my commits before making edits to the PR

Copy link
Contributor

@adidahiya adidahiya left a comment

Choose a reason for hiding this comment

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

looks good!

image

@adidahiya adidahiya changed the title [icons] feat: new icon - rocket [icons] feat: new icon "rocket" Aug 16, 2021
@adidahiya adidahiya merged commit 79ee10b into palantir:develop Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants