Skip to content

chore(icons,icons-ui): fetch icons from new @spectrum-css/ui-icons package #3857

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

Merged
merged 3 commits into from
Dec 19, 2023

Conversation

castastrophe
Copy link
Contributor

@castastrophe castastrophe commented Dec 6, 2023

Description

As part of a clean-up effort, Spectrum CSS is moving SVG processing for the UI icons library out of the @spectrum-css/icon styles package and into it's own @adobe/spectrum-css-ui-icons package.

This draft PR was tested against a linked local version and was successful. Once we release the new package on npm, we can migrate this to ready to review.

How has this been tested?

  • Validate no regressions in ui icons
    1. Set up a local yarn link for the ui-icons folder
    2. Updated SWC references from @spectrum-css/icon to @adobe/spectrum-css-ui-icons
    3. Run the build and validate the results

Types of changes

  • Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Checklist

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • If my change required a change to the documentation, I have updated the documentation in this pull request.
  • I have read the CONTRIBUTING document.
  • All new and existing tests passed.
  • I have reviewed at the Accessibility Practices for this feature, see: Aria Practices

Best practices

This repository uses conventional commit syntax for each commit message; note that the GitHub UI does not use this by default so be cautious when accepting suggested changes. Avoid the "Update branch" button on the pull request and opt instead for rebasing your branch against main.

@castastrophe castastrophe changed the title feat(icons,icons-ui): fetch icons from new @adobe/spectrum-css-ui-icons package chore(icons,icons-ui): fetch icons from new @adobe/spectrum-css-ui-icons package Dec 6, 2023
@castastrophe castastrophe changed the title chore(icons,icons-ui): fetch icons from new @adobe/spectrum-css-ui-icons package chore(icons,icons-ui): fetch icons from new @spectrum-css/ui-icons package Dec 7, 2023
@Westbrook Westbrook force-pushed the feat-update-icon-package-source branch from 80b6a93 to c26ebfb Compare December 19, 2023 21:50
@Westbrook Westbrook marked this pull request as ready for review December 19, 2023 22:10
Copy link
Contributor

@Westbrook Westbrook left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for fast forwarding this update for us :shipit:

@Westbrook Westbrook merged commit d6ac34f into main Dec 19, 2023
@Westbrook Westbrook deleted the feat-update-icon-package-source branch December 19, 2023 22:10
Copy link

github-actions bot commented Dec 19, 2023

Tachometer results

Chrome

accordion permalink

Version Bytes Avg Time vs remote vs branch
npm latest 418 kB 170.53ms - 175.70ms - unsure 🔍
-2% - +1%
-3.80ms - +2.43ms
branch 409 kB 172.07ms - 175.53ms unsure 🔍
-1% - +2%
-2.43ms - +3.80ms
-

action-bar permalink

Version Bytes Avg Time vs remote vs branch
npm latest 487 kB 92.14ms - 93.88ms - unsure 🔍
-1% - +1%
-0.78ms - +1.27ms
branch 478 kB 92.22ms - 93.31ms unsure 🔍
-1% - +1%
-1.27ms - +0.78ms
-

action-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 515 kB 114.47ms - 116.55ms - unsure 🔍
-1% - +1%
-1.73ms - +1.00ms
branch 506 kB 114.99ms - 116.76ms unsure 🔍
-1% - +2%
-1.00ms - +1.73ms
-

action-group permalink

Version Bytes Avg Time vs remote vs branch
npm latest 536 kB 71.05ms - 72.26ms - unsure 🔍
-2% - +0%
-1.61ms - +0.24ms
branch 527 kB 71.64ms - 73.04ms unsure 🔍
-0% - +2%
-0.24ms - +1.61ms
-

action-menu permalink

Version Bytes Avg Time vs remote vs branch
npm latest 636 kB 184.01ms - 190.63ms - unsure 🔍
-1% - +3%
-2.59ms - +4.71ms
branch 629 kB 184.73ms - 187.80ms unsure 🔍
-3% - +1%
-4.71ms - +2.59ms
-

alert-dialog permalink

Version Bytes Avg Time vs remote vs branch
npm latest 456 kB 127.21ms - 127.94ms - faster ✔
0% - 2%
0.24ms - 2.15ms
branch 447 kB 127.88ms - 129.66ms slower ❌
0% - 2%
0.24ms - 2.15ms
-

button-group permalink

Version Bytes Avg Time vs remote vs branch
npm latest 443 kB 63.38ms - 64.41ms - unsure 🔍
-1% - +1%
-0.92ms - +0.64ms
branch 434 kB 63.44ms - 64.62ms unsure 🔍
-1% - +1%
-0.64ms - +0.92ms
-

button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 449 kB 79.37ms - 81.12ms - unsure 🔍
-0% - +2%
-0.38ms - +1.69ms
branch 440 kB 79.04ms - 80.15ms unsure 🔍
-2% - +0%
-1.69ms - +0.38ms
-

card permalink

Version Bytes Avg Time vs remote vs branch
npm latest 509 kB 68.93ms - 72.34ms - unsure 🔍
-3% - +3%
-1.86ms - +2.30ms
branch 500 kB 69.23ms - 71.61ms unsure 🔍
-3% - +3%
-2.30ms - +1.86ms
-

checkbox permalink

Version Bytes Avg Time vs remote vs branch
npm latest 421 kB 98.91ms - 100.10ms - slower ❌
0% - 2%
0.04ms - 1.57ms
branch 412 kB 98.22ms - 99.18ms faster ✔
0% - 2%
0.04ms - 1.57ms
-

dialog permalink

Version Bytes Avg Time vs remote vs branch
npm latest 508 kB 88.77ms - 89.64ms - unsure 🔍
-0% - +1%
-0.35ms - +0.68ms
branch 499 kB 88.76ms - 89.32ms unsure 🔍
-1% - +0%
-0.68ms - +0.35ms
-

field-label permalink

Version Bytes Avg Time vs remote vs branch
npm latest 393 kB 25.62ms - 26.05ms - unsure 🔍
-0% - +2%
-0.12ms - +0.41ms
branch 384 kB 25.53ms - 25.84ms unsure 🔍
-2% - +0%
-0.41ms - +0.12ms
-

infield-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 405 kB 21.08ms - 21.32ms - unsure 🔍
-1% - +1%
-0.12ms - +0.20ms
branch 396 kB 21.05ms - 21.27ms unsure 🔍
-1% - +1%
-0.20ms - +0.12ms
-

menu permalink

Version Bytes Avg Time vs remote vs branch
npm latest 464 kB 246.71ms - 252.75ms - faster ✔
6% - 9%
16.68ms - 24.49ms
branch 456 kB 267.83ms - 272.79ms slower ❌
7% - 10%
16.68ms - 24.49ms
-

meter permalink

Version Bytes Avg Time vs remote vs branch
npm latest 408 kB 64.23ms - 64.90ms - unsure 🔍
-1% - +1%
-0.57ms - +0.38ms
branch 400 kB 64.32ms - 65.00ms unsure 🔍
-1% - +1%
-0.38ms - +0.57ms
-

number-field permalink

Version Bytes Avg Time vs remote vs branch
npm latest 520 kB 106.37ms - 107.84ms - unsure 🔍
-2% - +1%
-1.97ms - +0.78ms
branch 509 kB 106.54ms - 108.86ms unsure 🔍
-1% - +2%
-0.78ms - +1.97ms
-

overlay permalink

Version Bytes Avg Time vs remote vs branch
npm latest 511 kB 57.93ms - 58.92ms - unsure 🔍
-3% - +1%
-1.77ms - +0.89ms
branch 511 kB 57.63ms - 60.10ms unsure 🔍
-2% - +3%
-0.89ms - +1.77ms
-

picker-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 417 kB 47.81ms - 48.25ms - unsure 🔍
-1% - +1%
-0.40ms - +0.26ms
branch 408 kB 47.85ms - 48.34ms unsure 🔍
-1% - +1%
-0.26ms - +0.40ms
-

picker permalink

Version Bytes Avg Time vs remote vs branch
npm latest 506 kB 630.48ms - 639.20ms - unsure 🔍
-1% - +1%
-4.07ms - +7.95ms
branch 499 kB 628.77ms - 637.04ms unsure 🔍
-1% - +1%
-7.95ms - +4.07ms
-

popover permalink

Version Bytes Avg Time vs remote vs branch
npm latest 384 kB 22.68ms - 22.86ms - unsure 🔍
-0% - +1%
-0.10ms - +0.14ms
branch 375 kB 22.67ms - 22.83ms unsure 🔍
-1% - +0%
-0.14ms - +0.10ms
-

progress-bar permalink

Version Bytes Avg Time vs remote vs branch
npm latest 409 kB 51.62ms - 52.26ms - unsure 🔍
-1% - +1%
-0.32ms - +0.64ms
branch 400 kB 51.43ms - 52.14ms unsure 🔍
-1% - +1%
-0.64ms - +0.32ms
-

search permalink

Version Bytes Avg Time vs remote vs branch
npm latest 474 kB 62.62ms - 63.44ms - unsure 🔍
-1% - +0%
-0.93ms - +0.18ms
branch 466 kB 63.03ms - 63.77ms unsure 🔍
-0% - +1%
-0.18ms - +0.93ms
-

slider permalink

Version Bytes Avg Time vs remote vs branch
npm latest 476 kB 120.93ms - 124.21ms - unsure 🔍
-0% - +3%
-0.28ms - +3.44ms
branch 464 kB 120.11ms - 121.87ms unsure 🔍
-3% - +0%
-3.44ms - +0.28ms
-

split-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 711 kB 1853.91ms - 1856.38ms - unsure 🔍
-0% - +0%
-2.57ms - +0.99ms
branch 703 kB 1854.65ms - 1857.22ms unsure 🔍
-0% - +0%
-0.99ms - +2.57ms
-

swatch permalink

Version Bytes Avg Time vs remote vs branch
npm latest 410 kB 25.33ms - 25.60ms - slower ❌
0% - 2%
0.02ms - 0.40ms
branch 401 kB 25.12ms - 25.39ms faster ✔
0% - 2%
0.02ms - 0.40ms
-

switch permalink

Version Bytes Avg Time vs remote vs branch
npm latest 402 kB 27.29ms - 27.51ms - unsure 🔍
-1% - +0%
-0.31ms - +0.04ms
branch 393 kB 27.40ms - 27.67ms unsure 🔍
-0% - +1%
-0.04ms - +0.31ms
-

table permalink

Version Bytes Avg Time vs remote vs branch
npm latest 537 kB 325.24ms - 329.67ms - unsure 🔍
-2% - +1%
-5.38ms - +1.76ms
branch 504 kB 326.46ms - 332.07ms unsure 🔍
-1% - +2%
-1.76ms - +5.38ms
-

tabs permalink

Version Bytes Avg Time vs remote vs branch
npm latest 426 kB 122.44ms - 123.80ms - unsure 🔍
-1% - +1%
-1.40ms - +0.65ms
branch 418 kB 122.73ms - 124.26ms unsure 🔍
-1% - +1%
-0.65ms - +1.40ms
-

tags permalink

Version Bytes Avg Time vs remote vs branch
npm latest 456 kB 39.32ms - 42.34ms - slower ❌
81% - 95%
17.61ms - 20.63ms
branch 448 kB 21.63ms - 21.79ms faster ✔
45% - 49%
17.61ms - 20.63ms
-

textfield permalink

Version Bytes Avg Time vs remote vs branch
npm latest 433 kB 40.33ms - 41.59ms - unsure 🔍
-2% - +2%
-0.87ms - +0.96ms
branch 424 kB 40.24ms - 41.58ms unsure 🔍
-2% - +2%
-0.96ms - +0.87ms
-

toast permalink

Version Bytes Avg Time vs remote vs branch
npm latest 436 kB 48.47ms - 49.03ms - unsure 🔍
-1% - +1%
-0.53ms - +0.31ms
branch 427 kB 48.55ms - 49.18ms unsure 🔍
-1% - +1%
-0.31ms - +0.53ms
-

tooltip permalink

Version Bytes Avg Time vs remote vs branch
npm latest 574 kB 50.69ms - 51.42ms - unsure 🔍
-3% - +0%
-1.51ms - +0.08ms
branch 546 kB 51.07ms - 52.47ms unsure 🔍
-0% - +3%
-0.08ms - +1.51ms
-

top-nav permalink

Version Bytes Avg Time vs remote vs branch
npm latest 431 kB 84.78ms - 85.90ms - unsure 🔍
-1% - +1%
-0.88ms - +0.73ms
branch 423 kB 84.83ms - 86.00ms unsure 🔍
-1% - +1%
-0.73ms - +0.88ms
-
Firefox

accordion permalink

Version Bytes Avg Time vs remote vs branch
npm latest 418 kB 321.40ms - 330.76ms - unsure 🔍
-3% - +3%
-10.32ms - +8.60ms
branch 409 kB 318.72ms - 335.16ms unsure 🔍
-3% - +3%
-8.60ms - +10.32ms
-

action-bar permalink

Version Bytes Avg Time vs remote vs branch
npm latest 487 kB 175.91ms - 184.85ms - unsure 🔍
-5% - +2%
-8.50ms - +4.30ms
branch 478 kB 177.89ms - 187.07ms unsure 🔍
-2% - +5%
-4.30ms - +8.50ms
-

action-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 515 kB 280.52ms - 290.44ms - unsure 🔍
-3% - +3%
-7.61ms - +7.29ms
branch 506 kB 280.08ms - 291.20ms unsure 🔍
-3% - +3%
-7.29ms - +7.61ms
-

action-group permalink

Version Bytes Avg Time vs remote vs branch
npm latest 536 kB 182.83ms - 191.69ms - unsure 🔍
-3% - +4%
-5.73ms - +6.65ms
branch 527 kB 182.48ms - 191.12ms unsure 🔍
-4% - +3%
-6.65ms - +5.73ms
-

action-menu permalink

Version Bytes Avg Time vs remote vs branch
npm latest 636 kB 340.18ms - 351.14ms - unsure 🔍
-2% - +2%
-7.92ms - +6.36ms
branch 629 kB 341.87ms - 351.01ms unsure 🔍
-2% - +2%
-6.36ms - +7.92ms
-

alert-dialog permalink

Version Bytes Avg Time vs remote vs branch
npm latest 456 kB 223.63ms - 231.53ms - unsure 🔍
-3% - +2%
-6.35ms - +4.83ms
branch 447 kB 224.38ms - 232.30ms unsure 🔍
-2% - +3%
-4.83ms - +6.35ms
-

button-group permalink

Version Bytes Avg Time vs remote vs branch
npm latest 443 kB 170.21ms - 179.79ms - unsure 🔍
-2% - +5%
-2.68ms - +9.12ms
branch 434 kB 168.34ms - 175.22ms unsure 🔍
-5% - +1%
-9.12ms - +2.68ms
-

button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 449 kB 183.76ms - 189.40ms - unsure 🔍
-3% - +2%
-5.61ms - +3.25ms
branch 440 kB 184.34ms - 191.18ms unsure 🔍
-2% - +3%
-3.25ms - +5.61ms
-

card permalink

Version Bytes Avg Time vs remote vs branch
npm latest 509 kB 129.34ms - 134.38ms - unsure 🔍
-5% - +1%
-6.92ms - +1.80ms
branch 500 kB 130.86ms - 137.98ms unsure 🔍
-1% - +5%
-1.80ms - +6.92ms
-

checkbox permalink

Version Bytes Avg Time vs remote vs branch
npm latest 421 kB 198.32ms - 206.88ms - unsure 🔍
-3% - +3%
-5.57ms - +6.29ms
branch 412 kB 198.14ms - 206.34ms unsure 🔍
-3% - +3%
-6.29ms - +5.57ms
-

dialog permalink

Version Bytes Avg Time vs remote vs branch
npm latest 508 kB 138.51ms - 143.25ms - unsure 🔍
-5% - +1%
-6.96ms - +1.24ms
branch 499 kB 140.39ms - 147.09ms unsure 🔍
-1% - +5%
-1.24ms - +6.96ms
-

field-label permalink

Version Bytes Avg Time vs remote vs branch
npm latest 393 kB 67.95ms - 73.81ms - unsure 🔍
-7% - +5%
-4.82ms - +3.30ms
branch 384 kB 68.83ms - 74.45ms unsure 🔍
-5% - +7%
-3.30ms - +4.82ms
-

infield-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 405 kB 52.49ms - 55.95ms - unsure 🔍
-5% - +3%
-2.90ms - +1.50ms
branch 396 kB 53.56ms - 56.28ms unsure 🔍
-3% - +5%
-1.50ms - +2.90ms
-

menu permalink

Version Bytes Avg Time vs remote vs branch
npm latest 464 kB 437.36ms - 452.36ms - unsure 🔍
-1% - +4%
-4.46ms - +15.70ms
branch 456 kB 432.50ms - 445.98ms unsure 🔍
-4% - +1%
-15.70ms - +4.46ms
-

meter permalink

Version Bytes Avg Time vs remote vs branch
npm latest 408 kB 114.19ms - 122.17ms - unsure 🔍
-4% - +4%
-4.64ms - +5.12ms
branch 400 kB 115.13ms - 120.75ms unsure 🔍
-4% - +4%
-5.12ms - +4.64ms
-

number-field permalink

Version Bytes Avg Time vs remote vs branch
npm latest 520 kB 224.22ms - 231.30ms - unsure 🔍
-2% - +2%
-4.72ms - +5.40ms
branch 509 kB 223.81ms - 231.03ms unsure 🔍
-2% - +2%
-5.40ms - +4.72ms
-

overlay permalink

Version Bytes Avg Time vs remote vs branch
npm latest 582 kB 129.89ms - 134.75ms - unsure 🔍
-3% - +2%
-4.53ms - +2.93ms
branch 572 kB 130.29ms - 135.95ms unsure 🔍
-2% - +3%
-2.93ms - +4.53ms
-

picker-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 417 kB 84.02ms - 90.34ms - unsure 🔍
-4% - +6%
-3.60ms - +5.00ms
branch 408 kB 83.57ms - 89.39ms unsure 🔍
-6% - +4%
-5.00ms - +3.60ms
-

picker permalink

Version Bytes Avg Time vs remote vs branch
npm latest 506 kB 990.23ms - 1013.93ms - unsure 🔍
-0% - +2%
-4.29ms - +21.57ms
branch 499 kB 988.28ms - 998.60ms unsure 🔍
-2% - +0%
-21.57ms - +4.29ms
-

popover permalink

Version Bytes Avg Time vs remote vs branch
npm latest 384 kB 53.23ms - 57.49ms - unsure 🔍
-4% - +5%
-1.97ms - +2.81ms
branch 375 kB 53.85ms - 56.03ms unsure 🔍
-5% - +4%
-2.81ms - +1.97ms
-

progress-bar permalink

Version Bytes Avg Time vs remote vs branch
npm latest 409 kB 92.71ms - 98.89ms - unsure 🔍
-4% - +5%
-3.71ms - +4.47ms
branch 400 kB 92.74ms - 98.10ms unsure 🔍
-5% - +4%
-4.47ms - +3.71ms
-

search permalink

Version Bytes Avg Time vs remote vs branch
npm latest 474 kB 122.85ms - 129.19ms - unsure 🔍
-3% - +5%
-4.08ms - +5.72ms
branch 466 kB 121.46ms - 128.94ms unsure 🔍
-5% - +3%
-5.72ms - +4.08ms
-

slider permalink

Version Bytes Avg Time vs remote vs branch
npm latest 476 kB 224.10ms - 231.34ms - unsure 🔍
-2% - +2%
-4.08ms - +5.32ms
branch 464 kB 224.10ms - 230.10ms unsure 🔍
-2% - +2%
-5.32ms - +4.08ms
-

split-button permalink

Version Bytes Avg Time vs remote vs branch
npm latest 711 kB 1571.71ms - 1576.41ms - unsure 🔍
-0% - -0%
-5.97ms - -0.39ms
branch 703 kB 1575.73ms - 1578.75ms unsure 🔍
+0% - +0%
+0.39ms - +5.97ms
-

swatch permalink

Version Bytes Avg Time vs remote vs branch
npm latest 410 kB 69.50ms - 75.30ms - unsure 🔍
-9% - +1%
-7.23ms - +0.63ms
branch 401 kB 73.04ms - 78.36ms unsure 🔍
-1% - +10%
-0.63ms - +7.23ms
-

switch permalink

Version Bytes Avg Time vs remote vs branch
npm latest 402 kB 70.58ms - 78.18ms - unsure 🔍
-7% - +6%
-5.10ms - +4.18ms
branch 393 kB 72.19ms - 77.49ms unsure 🔍
-6% - +7%
-4.18ms - +5.10ms
-

table permalink

Version Bytes Avg Time vs remote vs branch
npm latest 537 kB 532.39ms - 543.73ms - unsure 🔍
-1% - +2%
-6.45ms - +10.09ms
branch 504 kB 530.22ms - 542.26ms unsure 🔍
-2% - +1%
-10.09ms - +6.45ms
-

tabs permalink

Version Bytes Avg Time vs remote vs branch
npm latest 426 kB 204.44ms - 211.64ms - unsure 🔍
-3% - +2%
-6.33ms - +4.45ms
branch 418 kB 204.97ms - 212.99ms unsure 🔍
-2% - +3%
-4.45ms - +6.33ms
-

tags permalink

Version Bytes Avg Time vs remote vs branch
npm latest 456 kB 46.70ms - 50.70ms - unsure 🔍
-4% - +7%
-2.05ms - +3.45ms
branch 448 kB 46.11ms - 49.89ms unsure 🔍
-7% - +4%
-3.45ms - +2.05ms
-

textfield permalink

Version Bytes Avg Time vs remote vs branch
npm latest 433 kB 79.76ms - 85.84ms - unsure 🔍
-8% - +3%
-6.53ms - +2.49ms
branch 424 kB 81.48ms - 88.16ms unsure 🔍
-3% - +8%
-2.49ms - +6.53ms
-

toast permalink

Version Bytes Avg Time vs remote vs branch
npm latest 436 kB 105.90ms - 112.86ms - unsure 🔍
-5% - +4%
-5.72ms - +4.12ms
branch 427 kB 106.70ms - 113.66ms unsure 🔍
-4% - +5%
-4.12ms - +5.72ms
-

tooltip permalink

Version Bytes Avg Time vs remote vs branch
npm latest 644 kB 104.37ms - 106.67ms - unsure 🔍
-2% - +1%
-2.00ms - +1.00ms
branch 635 kB 105.06ms - 106.98ms unsure 🔍
-1% - +2%
-1.00ms - +2.00ms
-

top-nav permalink

Version Bytes Avg Time vs remote vs branch
npm latest 431 kB 137.49ms - 145.87ms - unsure 🔍
-6% - +3%
-8.99ms - +5.07ms
branch 423 kB 138.00ms - 149.28ms unsure 🔍
-4% - +6%
-5.07ms - +8.99ms
-

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.

2 participants