Skip to content

fix(material): consistently use outlined input variant#2256

Merged
sdirix merged 1 commit into
eclipsesource:masterfrom
LukasBoll:fix/material_outlineed
Jan 24, 2024
Merged

fix(material): consistently use outlined input variant#2256
sdirix merged 1 commit into
eclipsesource:masterfrom
LukasBoll:fix/material_outlineed

Conversation

@LukasBoll

Copy link
Copy Markdown
Contributor

Use outlined input variant as the default to match Material UI

@netlify

netlify Bot commented Jan 24, 2024

Copy link
Copy Markdown

Deploy Preview for jsonforms-examples ready!

Name Link
🔨 Latest commit 0b5b756
🔍 Latest deploy log https://app.netlify.com/sites/jsonforms-examples/deploys/65b1028975e31200085ba662
😎 Deploy Preview https://deploy-preview-2256--jsonforms-examples.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@LukasBoll
LukasBoll force-pushed the fix/material_outlineed branch from 3d8eed5 to 2a52cb4 Compare January 24, 2024 12:08
@coveralls

coveralls commented Jan 24, 2024

Copy link
Copy Markdown

Coverage Status

coverage: 84.796%. remained the same
when pulling 0b5b756 on LukasBoll:fix/material_outlineed
into 1791e5c on eclipsesource:master.

Comment thread MIGRATION.md Outdated

### Material Renderers using Outlined Inputs

JSON Forms has been updated to use the `outlined` input variant as the default, aligning more closely with the default style of Material UI.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
JSON Forms has been updated to use the `outlined` input variant as the default, aligning more closely with the default style of Material UI.
JSON Forms now uses the `outlined` input variant as the default, aligning with the default style of Material UI since version 5.

Comment thread MIGRATION.md Outdated
### Material Renderers using Outlined Inputs

JSON Forms has been updated to use the `outlined` input variant as the default, aligning more closely with the default style of Material UI.
To use the `standard` input variant, as in previous versions of JSON Forms, it is possible to provide JSON Forms with a theme provider:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
To use the `standard` input variant, as in previous versions of JSON Forms, it is possible to provide JSON Forms with a theme provider:
If you would like to use the `standard` input variant, as was default in previous versions of JSON Forms, then this can be accomplished using the Material UI `ThemeProvider`:

Use outlined input variant as the default to match Material UI
@LukasBoll
LukasBoll force-pushed the fix/material_outlineed branch from 2a52cb4 to 0b5b756 Compare January 24, 2024 12:28
@LukasBoll
LukasBoll requested a review from sdirix January 24, 2024 12:29

@sdirix sdirix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@sdirix
sdirix merged commit 3a68d0c into eclipsesource:master Jan 24, 2024
@sdirix sdirix linked an issue Jan 30, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade Material Renderer Inputs to be aware of Variant

3 participants