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

feat(Icons): add 'edit' icon to spritemap #1760

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Conversation

booc0mtaco
Copy link
Contributor

Summary:

This is added to sync with one mentioned in the foundation sprite sheet

Test Plan:

  • Wrote automated tests
  • CI tests / new tests are not applicable
  • Manually tested my changes, but I want to keep the details secret
  • Manually tested my changes, and here are the details:
    • Create an alpha publish and try out in edu-stack or traject as a sanity check if changes affect build or deploy, or are breaking, such as token changes, widely used component updates, hooks changes, and major dependency upgrades.

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #1760 (085dd8b) into next (e3f2bc1) will decrease coverage by 0.03%.
Report is 2 commits behind head on next.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             next    #1760      +/-   ##
==========================================
- Coverage   92.22%   92.19%   -0.03%     
==========================================
  Files         146      146              
  Lines        2584     2588       +4     
  Branches      664      666       +2     
==========================================
+ Hits         2383     2386       +3     
- Misses        200      201       +1     
  Partials        1        1              
Files Changed Coverage Δ
src/components/Breadcrumbs/Breadcrumbs.tsx 95.45% <ø> (ø)
src/components/Popover/Popover.tsx 91.89% <ø> (ø)
src/icons/spritemap.tsx 100.00% <ø> (ø)
src/components/Accordion/Accordion.tsx 96.49% <75.00%> (-1.63%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

size-limit report 📦

Path Size
components 95.48 KB (+0.02% 🔺)
styles 32.77 KB (0%)

@booc0mtaco
Copy link
Contributor Author

Screenshot 2023-09-19 at 13 21 59 Screenshot 2023-09-19 at 13 21 38

@booc0mtaco
Copy link
Contributor Author

Sadly, it looks like the names between design and eng are not in sync, so there will be some follow up work to name them similarly, and to add any missing ones where they belong.

@booc0mtaco booc0mtaco requested a review from a team September 19, 2023 18:24
@booc0mtaco
Copy link
Contributor Author

cc @allisonbrownczi

@booc0mtaco booc0mtaco merged commit f56983a into next Sep 19, 2023
6 checks passed
@booc0mtaco booc0mtaco deleted the aholloway/EDS-1109 branch September 19, 2023 22:09
@booc0mtaco booc0mtaco mentioned this pull request Sep 22, 2023
booc0mtaco added a commit that referenced this pull request Sep 22, 2023
## [13.3.0](v13.2.0...v13.3.0) (2023-09-22)


### Features

* add event hooks to existing components ([#1757](#1757)) ([9045cf5](9045cf5))
* **Icons:** add 'edit' icon to spritemap ([#1760](#1760)) ([f56983a](f56983a))
* **Select:** add warning when name is missing from component props ([#1756](#1756)) ([e3f2bc1](e3f2bc1))


### Bug Fixes

* remove deprecated tokens from wireframe page ([#1752](#1752)) ([5e388fa](5e388fa))
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