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

[Contact Page] Add c-input modifier for more padding #2094

Closed
tylersticka opened this issue Dec 1, 2022 · 1 comment
Closed

[Contact Page] Add c-input modifier for more padding #2094

tylersticka opened this issue Dec 1, 2022 · 1 comment
Assignees
Labels
🎨 design Requires visual, UX or UI design decisions

Comments

@tylersticka
Copy link
Member

In our prototype contact page, there is a textarea that is prefilled with multi-line content. The padding of the textarea works well for textareas that are adjacent to other form fields, but on its own with multiple lines of content, it feels a little off balance.

Current Proposed Modifier
Image Image
@tylersticka tylersticka self-assigned this Dec 1, 2022
@tylersticka tylersticka added the 🎨 design Requires visual, UX or UI design decisions label Dec 1, 2022
@tylersticka
Copy link
Member Author

After futzing with a few solutions, I don't know that I think this belongs on the component.

It's really only an issue when there is a lone textarea that happens to include multiple line breaks in it to begin with, which feels very… niche. And niche design changes are what utility classes are for. And we already have padding classes.

I'm going to close this and say we should use a padding utility for now.

@tylersticka tylersticka closed this as not planned Won't fix, can't repro, duplicate, stale Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 design Requires visual, UX or UI design decisions
Projects
None yet
Development

No branches or pull requests

1 participant