-
Notifications
You must be signed in to change notification settings - Fork 76
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
Calcite-Tree: drag-and-drop #2210
Comments
Needs to be re-scoped |
Reallocating to the |
Based on current Tree behaviors, I think this can be rescoped to improving hover states and adding drag-and-drop. |
Hover states are covered in issue #6930. Drag-and-drop is this only thing left to address from this issue. |
We should consider what the bounds of calcite-tree and calcite-list are. If we add drag and drop sorting support to tree, at what point would tree and list differentiate? They are starting to overlap in functionality and UI. Currently, calcite-list uses a tree-grid role and does sorting and nesting as well as having the actions at the end. cc @jcfranco |
I agree w/ @driskull about defining the bounds of both components since they are getting pretty similar. From the design-perspective, are there any guidelines that could help keep them distinct? Also, can we reach out to the Dashboards team about this request and whether |
Moving to the |
Description
As part of supporting responsive layouts in Dashboards, we are introducing a new UI surface for manipulating elements. This surface is essentially an expandable/collapsable tree structure that acts as an entry point for a number of editing workflows. The best component for this use case is
calcite-tree
, but it needs several enhancements to fully meet our needs.Acceptance Criteria
Highlight style hover state
Expandable menu example
Drag-and-drop example
Relevant Info
Comprehensive designs can be found here, especially on the Layout Tree and Drag-and-Drop pages: https://www.figma.com/file/zmHT65VqXpldbZy4cvjZlw/?node-id=213%3A21129
Which Component
calcite-tree
Example Use Case
The text was updated successfully, but these errors were encountered: