-
Notifications
You must be signed in to change notification settings - Fork 7
Extend theming options of Button
s and form fields (#210)
#212
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
Conversation
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.
Tried to also check CSS, but It is too complicated for me, so I do not garentee anything
|
||
### Basic Theming | ||
|
||
The following theme options define basic appearance of form fields. The options |
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.
Not sure if The options
is correct. I think that the
should be omit
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.
My feeling is it's rather correct, it relates to the options mentioned in the previous sentence.
Button
and "new" form fields (TextField
,TextArea
,SelectField
) theming options to be dynamically generated based on SCSS settings.--rui-form-field-border-color
--rui-form-field-hover-border-color
--rui-form-field-focus-border-color
--rui-form-field-filled-disabled-color
,--rui-form-field-outline-disabled-box-shadow
,--rui-form-field-valid-default-background
options added etc.--rui-form-field-<SIZE>-padding-vertical
and--rui-form-field-<SIZE>-padding-horizontal
to--rui-form-field-<SIZE>-padding-y
and--rui-form-field-<SIZE>-padding-x
.Button
theming options:--rui-button-disabled-opacity
--rui-button-disabled-cursor
--rui-button-group-filled-separator-color
Closes #210.