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

[Feature] Update suomifi-icons and suomifi-design-tokens dependencies #410

Merged
merged 7 commits into from
Nov 13, 2020

Conversation

LJKaski
Copy link
Collaborator

@LJKaski LJKaski commented Nov 12, 2020

Description

There were some recent updates and changes to the suomifi-icons and suomifi-design-tokens. This PR implements those changes to the ui-components.

Most noticeable changes:

  • Two new icons added
  • Changes to some color tokens, such as successBase
  • Two new color tokens, depthDark2 and 3, added

depthDark2 was implemented in the placeholder texts of input elements and depthDark3 replaced the border color in form elements.

Motivation and Context

The library needs to stay up to date with the changes in the style guide.

How Has This Been Tested?

Tested by automated tests and by running locally. Should still be tested in different projects and environments, since some dependencies were changed in the underlying libraries.

Release notes

Breaking change: changes to color token values
Other changes: new tokens and icons due to updated suomifi-icons and suomifi-design-tokens

@LJKaski LJKaski added the enhancement New feature or request label Nov 12, 2020
@LJKaski LJKaski self-assigned this Nov 12, 2020
ketsappi
ketsappi previously approved these changes Nov 13, 2020
ketsappi
ketsappi previously approved these changes Nov 13, 2020
@LJKaski LJKaski merged commit 1a126b7 into develop Nov 13, 2020
@aappoalander aappoalander deleted the feature/update-icons-and-tokens branch December 4, 2020 08:26
@aappoalander aappoalander mentioned this pull request Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants