Skip to content

Restore viewBox for new icons that were over-optimized (fixes #113)#116

Merged
stephaniehobson merged 2 commits intomainfrom
restore-icon-viewboxes
Aug 13, 2025
Merged

Restore viewBox for new icons that were over-optimized (fixes #113)#116
stephaniehobson merged 2 commits intomainfrom
restore-icon-viewboxes

Conversation

@maureenlholland
Copy link
Collaborator

@maureenlholland maureenlholland commented Aug 12, 2025

Some new icons were manually cleaned and have viewBoxes

  • arrow-down/left/up/right
  • caret-down/up
  • white variants

SVGO-optimized icons were removing viewBox by default (this is no longer an issue with SVGO v4)

  • all icons have viewbox that match their width and height attributes (for most this is "0 0 16 16" but the narrow variants will have a different width and 3rd viewbox value)

@maureenlholland maureenlholland linked an issue Aug 12, 2025 that may be closed by this pull request
@maureenlholland maureenlholland mentioned this pull request Aug 13, 2025
10 tasks
Copy link
Contributor

@stephaniehobson stephaniehobson left a comment

Choose a reason for hiding this comment

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

R+ 🔍

@stephaniehobson stephaniehobson merged commit 0ee8d63 into main Aug 13, 2025
@stephaniehobson stephaniehobson deleted the restore-icon-viewboxes branch August 13, 2025 20:44
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.

Add viewbox back to updated icons

2 participants