Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Update search form styling #398

Merged
merged 2 commits into from
Sep 19, 2023
Merged

Update search form styling #398

merged 2 commits into from
Sep 19, 2023

Conversation

carolinan
Copy link
Contributor

Description
Updates the default border radius for the button and input field.
Removes the font weight.

Closes #389

Screenshots

Testing Instructions
Add a search block with different settings to confirm that the design is correct ( see the issue ).

In the block's sidebar settings, adjust the border radius and confirm that the user's settings still overrides the default.

"elements": {
"button": {
"border": {
"radius": { "ref": "styles.elements.button.border.radius" }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This will keep the radius up to date if we ever decide to change the button border radius.
Note that the CSS field does not work the same way, there the radius is hardcoded.

Copy link
Contributor Author

@carolinan carolinan Sep 18, 2023

Choose a reason for hiding this comment

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

This is probably an editor bug, The border radius should already be inherited from the element button styles even without this code.
But I noticed that without this code, it only had the border radius on the front, not in the editor.

@carolinan carolinan mentioned this pull request Sep 18, 2023
53 tasks
@carolinan
Copy link
Contributor Author

carolinan commented Sep 18, 2023

Login/out and comment form:
I added a small amount of CSS to theme.json. If we must re-style the login form button, it will take more CSS and it may be better to add that to style.css.
forms with input fields with border radius

@MaggieCabrera MaggieCabrera merged commit 3ec7b47 into trunk Sep 19, 2023
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 this pull request may close these issues.

Improve search form appearance
2 participants