Skip to content

Update @mozilla-protocol/assets to 2.1.1 (#479)#480

Closed
stephaniehobson wants to merge 1 commit intomozilla:masterfrom
stephaniehobson:assets
Closed

Update @mozilla-protocol/assets to 2.1.1 (#479)#480
stephaniehobson wants to merge 1 commit intomozilla:masterfrom
stephaniehobson:assets

Conversation

@stephaniehobson
Copy link
Copy Markdown
Contributor

Description

Describe what this change does.

  • I have documented this change in the design system.
  • I have recorded this change in CHANGELOG.md.

Issue

Fix #479

Testing

  • All icons that were inside the ui folder have been moved
  • Background-size declarations have been added to places icons are used (new icon files are larger)
  • Black icons no longer have -black in the file name.

@stephaniehobson
Copy link
Copy Markdown
Contributor Author

@alexgibson Could you have a look at the package-lock file please? It added a bunch of stuff as "optional" when I ran the upgrade.

@stephaniehobson stephaniehobson added P1 First level priority - Must have Needs:Review 👋 Ready for Developer Review Review: XS Code review time: 30 mins or less labels Sep 9, 2019
@alexgibson
Copy link
Copy Markdown
Contributor

@stephaniehobson try updating the version of npm you have installed locally on your machine.

@stephaniehobson
Copy link
Copy Markdown
Contributor Author

The package-lock file is co-operating now :)

This is ready for a general code review.

@stephaniehobson
Copy link
Copy Markdown
Contributor Author

Closing this in favour of going straight to 3.0.0.

@stephaniehobson stephaniehobson removed the Needs:Review 👋 Ready for Developer Review label Oct 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P1 First level priority - Must have Review: XS Code review time: 30 mins or less

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update @mozilla-protocol/assets to 3.0.0

2 participants