Skip to content
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

Vanilla styles - Automatically use default styles, remove redux style support, support style def merging #1675

Merged
merged 3 commits into from
Jan 18, 2021

Conversation

lucas-koehler
Copy link
Contributor

Apply default vanilla styles if none are explicitly specified

  • Set vanillaStyles as default value in JsonFormsStyleContext
  • Move vanillaStyles, StyleDef interface and the styles context to new module styles
  • Adapt documentation

Remove vanilla style support via redux store

Vanilla renderer style definitions can no longer be provided via the redux store.
Instead, the plain react JsonFormsStyleContext needs to be used.

  • Adapt style context
  • Adapt tests
  • Extend migration documentation

Merge vanilla styles with same id

  • adapt findStyle method
  • extend Styles documentation

* Set vanillaStyles as default value in JsonFormsStyleContext
* Move vanillaStyles, StyleDef interface and the styles context to new module styles
* Adapt documentation
Vanilla renderer style definitions can no longer be provided via the redux store.
Instead, the plain react JsonFormsStyleContext needs to be used.

* Adapt style context
* Adapt tests
* Extend migration documentation
@coveralls
Copy link

coveralls commented Jan 14, 2021

Coverage Status

Coverage decreased (-0.06%) to 88.455% when pulling b154063 on vanilla-styles into f07b6be on master.

packages/vanilla/Styles.md Outdated Show resolved Hide resolved
* adapt findStyle method
* extend Styles documentation
Copy link
Member

@sdirix sdirix left a comment

Choose a reason for hiding this comment

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

LGTM

@sdirix sdirix merged commit 569b23b into master Jan 18, 2021
@sdirix sdirix deleted the vanilla-styles branch January 18, 2021 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants