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
Added the styles for the new gutenberg-based search widget #1647
Conversation
…to gutenberg-search-form
.wp-block-search { | ||
.wp-block-search__label { | ||
@if variable-exists('bootstrap4') { | ||
@include sr-only(); |
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.
I think @extend would give smaller CSS.
@if variable-exists('bootstrap4') { | ||
@include sr-only(); | ||
} @else { | ||
@include visually-hidden(); |
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.
I think @extend would give smaller CSS.
@extend .input-group; | ||
|
||
.wp-block-search__input { | ||
@extend .form-control; |
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.
@@ -190,10 +194,17 @@ a.skip-link { | |||
margin-left: 0; | |||
} | |||
|
|||
@extend .btn-primary; | |||
@if variable-exists('bootstrap4') { | |||
@include button-variant( $primary, $primary ); |
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.
@extend .btn-primary;
gives marginally smaller CSS
margin-left: 0; | ||
} | ||
|
||
@if variable-exists('bootstrap4') { |
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.
Can be simplified to
@include button-variant( $primary, $primary );
@if variable-exists('bootstrap4') {
border-top-left-radius: 0;
border-bottom-left-radius: 0;
}
Description
Motivation and Context
Types of changes
Checklist:
develop
.develop
.composer cs:check
has passed locally.composer lint:php
has passed locally.Related Issues or Roadmap requests
Further comments