Skip to content

Shorthand supports for dash-core-components #1747

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

Closed
wants to merge 90 commits into from

Conversation

workaholicAB
Copy link
Contributor

@workaholicAB workaholicAB commented Sep 6, 2021

Replaced by #1763 and this can be closed, looks like too much commits

This PR implements this issue to support optional shorthand supports for below components.
You can compare old format and new format.

Dependency Update

  • Input Output State can now accept component object as well as string value of the component key
    • (Required) A separate PR will need to be merged before this

Updated Components with change notes

  • Dropdown

    • options props can now be passed without being a keyword argument
    • options can now accept array of string as well as previous format (array of dictionary)
  • RangeSlider

    • Changed props order to make it start with min max step
  • Checklist

    • options props becomes first argument, can be passed without keyword
    • Added inline props to specify if options are array of string or dictionary (previous format)
  • RadioItems

    • options props becomes first argument, can be passed without keyword
    • Added inline props to specify if options are array of string or dictionary (previous format)
  • DataTable

    • data and columns props becomes first 2 arguments, can be passed without keyword
    • (Optional) Separate PR will be followed to extract columns automatically from given data

@workaholicAB workaholicAB marked this pull request as draft September 18, 2021 18:34
@workaholicAB
Copy link
Contributor Author

@chriddyp Can we close this and move on to #1763? Or do we need to keep all review comments & histories?

@workaholicAB workaholicAB deleted the dcc/fix-issue960 branch September 20, 2021 20:57
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.

6 participants