This repository has been archived by the owner on Apr 13, 2022. It is now read-only.
Add color theme for 6 River MFP UI design system #179
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For now, this is the same as the base Polaris color theme except for the `primary` color role.
|
Hey Adam, if this works for you, go ahead and ship it! Mentioning it in the changelog would be great, though. |
kaelig
approved these changes
Feb 24, 2021
|
Super. I think I updated the changelog and incremented the version correctly. 🤞 🚢🚢🚢 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Much like the Retail team, we at 6 River Systems are building our own design system to address the needs of users of our devices in the warehouse environment.
Our product is called "MFP UI" (Mobile Fulfillment Platform User Interface), so the working name for our design system is "Polaris for MFP UI." Based on this, I chose the name
mfp-uifor this color theme. I'm not sure if that's correct, since the original color theme doesn't provide much to pattern off of (i.e. it's called "base" rather than "polaris-for-admin" or something like that). Please let me know if we should use a different name for this theme.Commits
For now, this is the same as the base Polaris color theme except for the
primarycolor role.