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

New design system applied to priority 1 components #523

Merged
merged 50 commits into from
Nov 9, 2022

Conversation

vmilan
Copy link
Contributor

@vmilan vmilan commented Nov 4, 2022

Description

Shortcut: #266115
[sc-266115]

Implement the new design system in priority 1 components. Toggle button will be in a different PR.

New Button component that uses Mui Button and wraps its children in Typography to meet with the designed behavior (text overflow case)

Also, stories and folders reorganization to meet the atomic design structure.

VictorVelarde and others added 30 commits October 19, 2022 19:41
@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #266115: COMPONENTS - Basic components (priority 1).

@vmilan vmilan changed the title New design system in priority 1 components New design system applied on priority 1 components Nov 4, 2022
@vmilan vmilan changed the title New design system applied on priority 1 components New design system applied to priority 1 components Nov 4, 2022
@vmilan vmilan marked this pull request as ready for review November 8, 2022 17:38
@vmilan vmilan requested a review from a team November 8, 2022 17:40
@@ -138,32 +137,32 @@ export const commonPalette = {
dark: baseColors.indigo[500],
light: baseColors.indigo[300],
contrastText: baseColors.common.white,
relatedDark: getMixedColor(baseColors.indigo[400], baseColors.neutral.A700, 0.6),
relatedLight: getMixedColor(baseColors.indigo[400], baseColors.common.white, 0.9)
relatedDark: '#0d2b4a',
Copy link
Contributor

Choose a reason for hiding this comment

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

nit, let's always try to keep uppercase

Copy link
Contributor

@VictorVelarde VictorVelarde left a comment

Choose a reason for hiding this comment

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

LGTM.
Like you said, folder restructuring / naming could have been in a separated PR, but I'm ok with landing this on feature branch, and I'll make an experimental release

@VictorVelarde VictorVelarde merged commit 4b7f247 into feature/design-system Nov 9, 2022
@VictorVelarde VictorVelarde deleted the feature/components-priority-1 branch November 9, 2022 18:30
@vmilan vmilan restored the feature/components-priority-1 branch November 10, 2022 08:19
@vmilan vmilan deleted the feature/components-priority-1 branch November 10, 2022 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants