Skip to content

Conversation

adamkudrna
Copy link
Member

@adamkudrna adamkudrna commented Jan 14, 2021

  • Refactored Button and "new" form fields (TextField, TextArea, SelectField) theming options to be dynamically generated based on SCSS settings.
  • Unified styling of form field priorities and validation states. Theming options removed in favour of variant/validation state specific options:
    • --rui-form-field-border-color
    • --rui-form-field-hover-border-color
    • --rui-form-field-focus-border-color
  • Extended theming options of aforementioned components with optional disabled interaction state options (Extend theming options for disabled buttons and form fields #210):
    • --rui-form-field-filled-disabled-color, --rui-form-field-outline-disabled-box-shadow, --rui-form-field-valid-default-background options added etc.
  • Renamed form field theming options --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.
  • Added Button theming options:
    • --rui-button-disabled-opacity
    • --rui-button-disabled-cursor
    • --rui-button-group-filled-separator-color
  • Documented everything above.
  • Minor fixes.

image
image
image
image
image
image

Closes #210.

@github-actions github-actions bot added the BC Breaking change label Jan 14, 2021
Copy link
Contributor

@bedrich-schindler bedrich-schindler left a 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
Copy link
Contributor

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

Copy link
Member Author

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.

@adamkudrna adamkudrna merged commit 239389a into master Jan 26, 2021
@adamkudrna adamkudrna deleted the bc/210 branch January 26, 2021 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BC Breaking change feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extend theming options for disabled buttons and form fields

3 participants