-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
dash/non-nested-element #21150
dash/non-nested-element #21150
Conversation
File size comparisonSizes for compiled+gzipped (bold) and compiled files.
|
Lighthouse reportdashboards-demo-minimal.json
|
Dashboard visual diffsNo differences found |
Visual test results - No difference found |
Benchmark report - Dashboardsbenchmarks/Dashboards/DataPool-CSV-constructor.bench.ts
See all
benchmarks/Dashboards/DataTable-loading-columns.bench.ts
See all
benchmarks/Dashboards/DataTable-loading-rows.bench.ts
See all
|
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.
Technically it looks & works ok! But I have two suggestions/ideas.
- I think we should write something about it in the EditMode docs article and maybe create a standalone sample?
- Maybe it would be good to apply this to some default options?
Hm..... We are adding some new concepts/wording here (nested, standalone) in addition to new option. I feel it's perhaps to verbose. Not possible with a simpler approach? E.g.
If the |
@stitot I'm fine with that but the only drawback is that with this approach it won't be possible to have collapsible singular elements like we used to have: |
Hm...... From a UX (and perhaps a11y?) perspective it makes sense for me to have all these single values exposed. Takes more space, but easier to find the value you try to edit (by looking at actual value, not name of field) instead of expanding them to see what's hidden inside. We could introduce an |
Added
isStandalone
option to add non-nested options in the edit mode sidebar, #20793.