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 icons "group-item", "input" #6573

Merged
merged 2 commits into from
Nov 29, 2023

Conversation

ju-won
Copy link
Contributor

@ju-won ju-won commented Nov 28, 2023

Fixes #0000

Checklist

  • Includes tests
  • Update documentation

Changes proposed in this pull request:

Added group and input icons

Reviewers should focus on:

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

Screenshot

image

@palantirtech
Copy link
Member

Thanks for your interest in palantir/blueprint, @ju-won! 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.

@adidahiya adidahiya changed the title added new icon(group, input) [icons] feat: new icons "(group, input) Nov 28, 2023
@adidahiya adidahiya changed the title [icons] feat: new icons "(group, input) [icons] feat: new icons "group", "input" Nov 28, 2023
@adidahiya adidahiya self-assigned this Nov 28, 2023
packages/icons/icons.json Outdated Show resolved Hide resolved
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.

lgtm in docs preview

image

@adidahiya adidahiya merged commit c3c1a47 into palantir:develop Nov 29, 2023
11 of 12 checks passed
@adidahiya adidahiya changed the title [icons] feat: new icons "group", "input" [icons] feat: new icons "group-item", "input" Nov 29, 2023
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