-
Notifications
You must be signed in to change notification settings - Fork 351
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
Remote: Comment form styles #5580
Conversation
f410934
to
1a3477f
Compare
Here is a corresponding wpcom patch D75679-code required to not break the comment form there. |
@kjellr do you think it's worth the custom CSS for this? |
1a3477f
to
da750d0
Compare
I think the CSS that styles the form fields is fine, but I would leave out the |
@kjellr this is the mockup: This is the styles implementation with grid: This is the syles implementation without grid: Should we remove the grid css? |
That's fine. We should fix the padding on the submit button too while we're at it. |
@kjellr sorry but I didn't get what is fine. Should we remove the grid CSS or not? |
Yes, please remove the grid. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
🚢
Changes proposed in this Pull Request:
Remote: Comment form styles
Related issue(s):
#5501