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

Update Shortcode design elements from design review #1810

Closed
chipsnyder opened this issue Jan 23, 2020 · 2 comments · Fixed by WordPress/gutenberg#19852
Closed

Update Shortcode design elements from design review #1810

chipsnyder opened this issue Jan 23, 2020 · 2 comments · Fixed by WordPress/gutenberg#19852
Assignees
Labels
Milestone

Comments

@chipsnyder
Copy link
Contributor

chipsnyder commented Jan 23, 2020

Describe the bug
From the design review with @iamthomasbishop:

  • Top Label:
    • Use system font "sans-serif" for ios
    • Set font color to be $gray in light mode and $gray-50 in dark mode
    • Increase margin between the label and input field

Screenshots

Screenshots

Placeholder

iOS

Default Dark mode
Placeholder-iOS iOS Dark - Placeholder
iPad Default iPad Dark mode
iPad - Placeholder iPad Dark - Placeholder

Android

Default
Android - Placeholder

Static

iOS

Default Dark mode
Static-iOS iOS Dark - Static
iPad Default iPad Dark mode
iPad - Static iPad Dark - Static

Android

Default
Android - Static

Selected

iOS

Default Dark mode
Selected-iOS iOS Dark - Selected
iPad Default iPad Dark mode
iPad - Selected iPad Dark - Selected

Android

Default
Android - Selected

Additional context
This is to make up for not performing design review before WordPress/gutenberg#19534 was merged

@chipsnyder chipsnyder added this to the 1.22 milestone Jan 23, 2020
@chipsnyder chipsnyder self-assigned this Jan 23, 2020
@iamthomasbishop
Copy link
Contributor

Thanks, Chip! To clarify by “Set font color to match $gray in light mode and $gray-50 in dark mode” I was referring to the top label (“Shortcode” above the input). I believe the placeholder text on the input was fine.

@iamthomasbishop
Copy link
Contributor

Increase margin between the label and input field

Fwiw, I think the spacing should be 4px, but it might be slightly different if the design is using a different line-height than the implementation.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants