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.

Sign up for GitHub

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

Jump to bottom

[IOAPPFD0-180] Update icon set with more than 20 new icons #121

Merged
merged 24 commits into from
Nov 6, 2023
Merged

[IOAPPFD0-180] Update icon set with more than 20 new icons #121

merged 24 commits into from
Nov 6, 2023

Conversation

dmnplb
Copy link
Collaborator

@dmnplb dmnplb commented Oct 30, 2023 •
edited
Loading

Warning

This PR depends on #118

Short description

This PR adds more than 20 new icons to the existing icon set. All the icons are generated through the new generateNewIcons process.

List of changes proposed in this pull request

  • Add more than 20 new icons (*.tsx extension and relative *.svg original file)

Preview

Note

New icons are marked with a green dot

How to test

  1. Launch the example app
  2. Go to the Icons page

dmnplb added 19 commits October 24, 2023 12:10
@dmnplb
Add the new generateNewIcon file (only comments for now) and relati…
12c213c 
…ve component template
@dmnplb
Add the AI-generated generateNewIcons file
1a6fbac
@dmnplb
Prettify the generated files
6bb2349
@dmnplb
Improve generateNewIcons script
16607f0
@dmnplb
Improve console log output with readable timestamp
9189205
@dmnplb
Disable some typescript errors at the top of the file
b687939
@dmnplb
Disable TS check in the generateNewTimestamp
567d4a3
@dmnplb
Merge branch 'main' into IOAPPFD0-179-draft-automation-processing-ico…
481ff02 
…ns-pictograms
@dmnplb
Delete useless variable declarations
c8f2402
@dmnplb
Add comment to the _IconTemplate.tsx file
96abf75
@dmnplb
Improve the script with correct SVG optimisation options, edit _IconT…
43cb85e 
…emplate
@dmnplb
Add the first draft of the new generateNewPictograms file
5b5a5e2
@dmnplb
Add correct colorValues
056c59e
@dmnplb
Improve the generateNewPictograms script using svgr
e383b6e
@dmnplb
Merge branch 'main' into IOAPPFD0-179-draft-automation-processing-ico…
08721f0 
…ns-pictograms
@dmnplb
Update generateNewIcons with svgr, add new icons
eb13285
@dmnplb
Update variable name to make it clearer
09d6e89
@dmnplb
Add new icons
454c5f1
@dmnplb
Add updated IconInbox… icons
b3f4398
@dmnplb dmnplb marked this pull request as ready for review October 30, 2023 10:12
@dmnplb dmnplb requested review from a team as code owners October 30, 2023 10:12
@dmnplb dmnplb added the Assets label Oct 30, 2023
@dpulls
Copy link

dpulls bot commented Oct 30, 2023

🎉 All dependencies have been resolved !

@dmnplb dmnplb merged commit 479ff90 into main Nov 6, 2023
2 checks passed
@dmnplb dmnplb deleted the IOAPPFD0-180-update-icon-set branch November 6, 2023 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CrisTofani CrisTofani CrisTofani approved these changes

Assignees
No one assigned
Labels
Assets
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dmnplb @CrisTofani