Directory

Improve search form appearance · Issue #389 · WordPress/twentytwentyfour · GitHub
Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Improve search form appearance #389

Closed
beafialho opened this issue Sep 17, 2023 · 8 comments · Fixed by #398
Closed

Improve search form appearance #389

beafialho opened this issue Sep 17, 2023 · 8 comments · Fixed by #398

Comments

@beafialho
Copy link

Unfortunately there's no UI to perfect these forms accurately but can we improve this? It's looking very unpolished. The placeholder font weight should be Regular and the input field border radius should be the same as the theme's buttons.

These details make a difference.

Reality Expectation
Captura de ecrã 2023-09-17, às 12 37 57 Captura de ecrã 2023-09-17, às 12 41 24
@hanneslsm
Copy link
Contributor

hanneslsm commented Sep 17, 2023

Changing the border radius of the input field is currently not possible: WordPress/gutenberg#51617 (on a global level)

@carolinan
Copy link
Contributor

carolinan commented Sep 18, 2023

Should all text input elements, like in the log in or comments, have the same radius?

@carolinan
Copy link
Contributor

Should only the placeholder text be regular, or also the text that the user enters into the field on the front?

@beafialho
Copy link
Author

Should all text input elements, like in the log in or comments, have the same radius?

All text inputs should ideally have the same radius, yes.

Should only the placeholder text be regular, or also the text that the user enters into the field on the front?

The placeholder and the text users enter should both be Regular.

@carolinan
Copy link
Contributor

carolinan commented Sep 18, 2023

Ouch the form for the login/out block is especially bad. I did not know that this button is missing the CSS class that styles the buttons.

Found an issue for it: WordPress/gutenberg#50466

form blocks

@beafialho
Copy link
Author

Can all input fields have the same border radius as the search?

@carolinan
Copy link
Contributor

I added a screenshot here: #398

@beafialho
Copy link
Author

Great, thank you!

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

Successfully merging a pull request may close this issue.

3 participants