Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Removing usage of hardcoded FINN blue colors and using branded versions #132

Merged
merged 10 commits into from
Oct 6, 2023

Conversation

ChristianTherkelsen
Copy link
Contributor

Why?

Enable branding for NMP

What?

  • Point to newest FinniversKit release
  • Using color branded image for verification badge
  • Replacing .primaryBlue and .secondaryBlue with color branded versions
  • Adding branded tintColor to collapsible component
  • Adding tintColor to favouriteButton image

Version Change

No breaking changes so this is a minor release

UI Changes

Showing all components that have been adjusted 👇 Finn version remain unchanged but Tori colors are now available for NMP

Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 40 40 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 40 11

Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 41 31 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 41 17

Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 41 57 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 41 46

Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 43 51 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 43 32

Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 33 12 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 33 10 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 33 34 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 33 33

Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 22 12 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 22 01 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 16 43 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 16 40

Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 13 00 33 Simulator Screenshot - iPhone 15 Pro - 2023-10-05 at 12 55 24

Copy link
Member

@bstien bstien left a comment

Choose a reason for hiding this comment

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

LGTM! Great job going through all of these views😄🥇

@ChristianTherkelsen
Copy link
Contributor Author

Thanks! 🙏 Sure takes a while to get through it all 😅

@ChristianTherkelsen ChristianTherkelsen merged commit 8f424da into master Oct 6, 2023
@ChristianTherkelsen ChristianTherkelsen deleted the nmp-brand-styling branch October 6, 2023 08:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants