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

Fix icon optimize config #10087

Merged
merged 1 commit into from
Aug 16, 2023
Merged

Fix icon optimize config #10087

merged 1 commit into from
Aug 16, 2023

Conversation

kyledurand
Copy link
Contributor

This was causing quite a bit of output in build scripts. Adding a width and height to an icon then running yarn optimize confirms that this still works

@kyledurand kyledurand added the 🤖Skip Changelog Causes CI to ignore changelog update check. label Aug 16, 2023
@kyledurand kyledurand merged commit 49f1257 into main Aug 16, 2023
12 of 13 checks passed
@kyledurand kyledurand deleted the fix-icon-optimize-config branch August 16, 2023 16:13
kyledurand added a commit that referenced this pull request Aug 17, 2023
This was causing quite a bit of output in build scripts. Adding a width
and height to an icon then running yarn optimize confirms that this
still works
yurm04 pushed a commit that referenced this pull request Aug 24, 2023
This was causing quite a bit of output in build scripts. Adding a width
and height to an icon then running yarn optimize confirms that this
still works
yurm04 pushed a commit that referenced this pull request Aug 24, 2023
This was causing quite a bit of output in build scripts. Adding a width
and height to an icon then running yarn optimize confirms that this
still works
sam-b-rose added a commit that referenced this pull request Aug 28, 2023
* main:
  Fix rename prop migration (#10215)
  Update translations: default (#10219)
  [Avatar] Override SE23 background override with imageHasLoaded (#10211)
  Create Code style guide.md (#10176)
  ActionsList filtering (#9907)
  Version Packages (#10078)
  Support dynamic fontsizes in iOS devices (#10181)
  Fix migration resolution path (#10188)
  [Button] Deprecate connectedDisclosure  (#10183)
  Update sandbox.md (#10186)
  Allow migrator to target subcomponents and change prop values (#10071)
  Remove `polaris-cli` and `polaris-codemods` packages (#10101)
  Bump loader-utils from 1.4.0 to 1.4.2 (#7725)
  Bump deep-object-diff from 1.1.7 to 1.1.9 (#7734)
  Bump decode-uri-component from 0.2.0 to 0.2.2 (#7832)
  Bump webpack from 5.72.0 to 5.76.0 (#8667)
  Bump fastify from 4.5.3 to 4.15.0 (#8750)
  [Modal] Activator no longer wrapped in Box (#10086)
  Fix icon optimize config (#10087)
  [ResourceList] Remove layout jank when bulk actions enabled and item selected
sam-b-rose added a commit that referenced this pull request Aug 28, 2023
* main:
  Fix rename prop migration (#10215)
  Update translations: default (#10219)
  [Avatar] Override SE23 background override with imageHasLoaded (#10211)
  Create Code style guide.md (#10176)
  ActionsList filtering (#9907)
  Version Packages (#10078)
  Support dynamic fontsizes in iOS devices (#10181)
  Fix migration resolution path (#10188)
  [Button] Deprecate connectedDisclosure  (#10183)
  Update sandbox.md (#10186)
  Allow migrator to target subcomponents and change prop values (#10071)
  Remove `polaris-cli` and `polaris-codemods` packages (#10101)
  Bump loader-utils from 1.4.0 to 1.4.2 (#7725)
  Bump deep-object-diff from 1.1.7 to 1.1.9 (#7734)
  Bump decode-uri-component from 0.2.0 to 0.2.2 (#7832)
  Bump webpack from 5.72.0 to 5.76.0 (#8667)
  Bump fastify from 4.5.3 to 4.15.0 (#8750)
  [Modal] Activator no longer wrapped in Box (#10086)
  Fix icon optimize config (#10087)
  [ResourceList] Remove layout jank when bulk actions enabled and item selected
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
This was causing quite a bit of output in build scripts. Adding a width
and height to an icon then running yarn optimize confirms that this
still works
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖Skip Changelog Causes CI to ignore changelog update check.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants