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

fix(select-field): spacing constants #159

Merged
merged 1 commit into from
May 9, 2024
Merged

fix(select-field): spacing constants #159

merged 1 commit into from
May 9, 2024

Conversation

frankcalise
Copy link
Contributor

Description

  • Fixes spacing.large constants to spacing.lg to reflect current Ignite styling better

@@ -688,12 +688,12 @@ function FavoriteNBATeamsScreen() {
<SelectField
label="NBA Team(s)"
helper="Select your team(s)"
placeholder="e.g. Trail Blazers"
placeholder="e.g. Knicks"
Copy link
Member

Choose a reason for hiding this comment

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

😂

Copy link
Member

@yulolimum yulolimum left a comment

Choose a reason for hiding this comment

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

@frankcalise
Copy link
Contributor Author

b8e7836#diff-2cfc2bcb77c9036757ea29f2f5c67a870df99f0b4cbb6bf9cb3cc1397e6b5394R691

@yulolimum technically Josh Hart is a Trail Blazer so we good

@mazenchami
Copy link

b8e7836#diff-2cfc2bcb77c9036757ea29f2f5c67a870df99f0b4cbb6bf9cb3cc1397e6b5394R691

@yulolimum technically Josh Hart is a Trail Blazer so we good

also technically technically, Knicks are in the playoffs 😁

@frankcalise frankcalise merged commit 1cbb101 into main May 9, 2024
1 check passed
@frankcalise frankcalise deleted the fix/select-field branch May 9, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants