-
Notifications
You must be signed in to change notification settings - Fork 62
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
IBX-612: As an Editor, I want to see redesigned input field in all states #1796
IBX-612: As an Editor, I want to see redesigned input field in all states #1796
Conversation
92438c6
to
1e0ed5a
Compare
.ibexa-form-error { | ||
display: flex; | ||
font-family: $font-family-sans-serif; | ||
font-size: calculateRem(12px); |
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.
$ibexa-text-font-size-small
?
Or maybe you could create some mixin because you repeat 3 time
font-family: $font-family-sans-serif;
font-size: calculateRem(12px);
letter-spacing: 0.12px;
line-height: calculateRem(18px);
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 was thinking about the same. But I'm not sure what I can combine to make it reusable.
Maybe 2 mixins:
- one with: font-family and letter-spacing
- and the other with: font-size and line-height
?
.ibexa-form-help { | ||
display: flex; | ||
font-family: $font-family-sans-serif; | ||
font-size: calculateRem(12px); |
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.
$ibexa-text-font-size-small
?
@include label-required(); | ||
|
||
font-family: $font-family-sans-serif; | ||
font-size: calculateRem(12px); |
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.
$ibexa-text-font-size-small
?
padding: 0; | ||
fill: $ibexa-color-dark; | ||
|
||
.ibexa-icon { |
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.
Maybe modifier in icons.scss ?
3635948
to
9be2fb3
Compare
@@ -40,10 +40,10 @@ | |||
} | |||
|
|||
&--text { | |||
font-family: $font-family-sans-serif; | |||
@include ibexa-font-sans-serif(); | |||
|
|||
font-size: $small-font-size; |
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.
Please use variable with "ibexa" prefix: $ibexa-text-font-size-small
or $ibexa-text-font-size-medium
src/bundle/Resources/views/themes/admin/ui/form_fields.html.twig
Outdated
Show resolved
Hide resolved
src/bundle/Resources/views/themes/admin/ui/form_fields.html.twig
Outdated
Show resolved
Hide resolved
88b204e
to
620b228
Compare
46bdd4a
to
2da6f37
Compare
75da991
to
c4bab88
Compare
c4bab88
to
9ca8189
Compare
…d ibexa-label--checkbox-radio
9ca8189
to
fec2b02
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Related PRs:
New look
New look --small
Checkbox/radio button labels
Checklist:
$ composer fix-cs
)