-
Notifications
You must be signed in to change notification settings - Fork 51
Zamoore/advanced table/column reordering 2 #2997
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
Merged
Merged
Changes from all commits
Commits
Show all changes
108 commits
Select commit
Hold shift + click to select a range
70d5559
add reorder components
zamoore 2ad6f29
picking reorder code
zamoore 788eef0
pulling in styling
zamoore 733baf9
set up initial showcase example
zamoore 4ed5a9f
working on styling
zamoore adcc00d
adding tests
zamoore cdf739b
working on adding tests for drag/drop behavior
zamoore 00ace6f
cleaning up visuals
zamoore 433e9f4
styling the reorder handle
zamoore 77a4e10
styling reorderable columns
zamoore f9c04c7
adding dropdown context options for reordering
zamoore a629f0c
working on context menu options for reordering
zamoore 66c79d2
can send column to first/last
zamoore 8238e27
applying new reorder handle styles
zamoore 6787efa
can step column with keyboard
zamoore fe43c94
keyboard movement works
zamoore c5698dd
added tests for context menu opptions
zamoore 00a31db
adding tests for context menu action execution
zamoore 67da9d3
added tests for reordering columns with focus/arrowkeys
zamoore b168775
styling srag target
zamoore 8facd98
fixed reorder handle styles
zamoore 9cbefdd
added styles for focus-within
zamoore c5f4f78
working on programatic focus for reorder handle
zamoore 310a7d8
added reorder capabilities to sortable columns
zamoore 19108ba
reordering resized columns preserves the column size
zamoore 6154a12
resizing reordered columns works
zamoore f92dbf0
making resizing and reordering work together
zamoore c89cbdb
working on width ledger
zamoore b2f2054
resizing works again but with a ledger system now
zamoore 688ca3e
reordering and resizing work together
zamoore 7488912
fix: reorder handle focus bug
b3c9b9b
merging main
zamoore 0b02296
fixing ts error
zamoore 5b80f06
fixed scroll stutter
zamoore 0c119a3
updated nontranslated string
zamoore 1b78bac
correctly calculating column width at drag start in order to construcβ¦
zamoore 61cbf7a
fixing drop target styles
zamoore a843397
fixed an issue with setting in nested rows
zamoore a40220f
adding a default column key (generated)
zamoore 3aa9d53
added an assertion for trying to use reorderable columns with nested β¦
zamoore 8ac2f35
got reorderable columns working with sticky first column
zamoore 1ba94a9
Merge branch 'main' into zamoore/AdvancedTable/column-reordering-2
zamoore 59758cd
made the demo reorderable
zamoore ad62203
adding showcase examples
zamoore d5959bc
adding showcase examples
zamoore 89a0568
fixed firefox reorder bug
zamoore 06e1fc1
wip target styles
zamoore 1bc2036
merge main
zamoore 8f66211
cleaning up PR and adding more tests
zamoore 4804a4d
moving columns with sticky columns works
zamoore 29113bf
prevent users from using keyboard nav to move column to sticky columnβ¦
zamoore 8d15dc5
adding tests around reordering and sticky columns
zamoore 65ea884
reverting a change to the gridTemplateColumns getter
zamoore 1d78f3c
not showing context menu if there are no reorder options
zamoore 8beaea8
cleaned up column order getter
zamoore 0b20575
fixed a bug where resizing reordered columns was resizing the wrong cβ¦
zamoore 367c2a2
merge main
zamoore 93d3f03
added some throttling to resizing to prevent lag
zamoore 0528abb
merged main
zamoore 44e972b
getting width repayment working with column reordering
zamoore aaca8f0
fixing width reset
zamoore 2566e2a
remove glint disable comment
zamoore 6ae6722
fixing linting errors
zamoore 2128db7
change the model type
zamoore 63c7331
fixing linting errors
zamoore 4e95dfb
fixing linting errors
zamoore ba38777
removing unnecessary type assertion
zamoore 555c480
Merge branch 'main' into zamoore/AdvancedTable/column-reordering-2
zamoore a1e060f
fixing a11y violation
zamoore 77f05dc
fixing tests
zamoore 6701fca
green tests
zamoore f6e38f7
added test waiter
zamoore 2d273a2
fixing broken unit tests
zamoore 18588f2
Merge branch 'main' into zamoore/AdvancedTable/column-reordering-2
zamoore 6afab77
increased the target for the reorder handle
zamoore 70a7cc8
changed order of th components
zamoore 26dc0f0
fixing linting error
zamoore b0a3c7d
responding to pr feedback
zamoore 09409fc
responding to pr feedback
zamoore 849ad53
addressing PR feedback
zamoore 13889dd
adding new test for context menu options for sticky col
zamoore 0022714
added an example with reorderable columns and selectable rows
zamoore 3bb1222
addressing PR feedback
zamoore b6bf61a
fixed dynamic segment in translation
zamoore 539f54a
correctly updates the column order when the value changes both internβ¦
zamoore eee4fad
Merge branch 'main' into zamoore/AdvancedTable/column-reordering-2
zamoore d40df90
not showing pin/unpin option if columns are reorderable
zamoore 793ec95
no longer showing 'move to first' option in context menu if there is β¦
zamoore 1a6495d
ammending tests
zamoore d644586
fixed the focus ring style on the reorder handle
zamoore be759d5
made sure the drop target had the correct styles when dropping on tabβ¦
zamoore 9d4c011
removed support for hasStickyFirstColumn together with hasReorderableβ¦
zamoore 4e5cc9d
fixed assertion test
zamoore a3b35ee
removed column reordering from the generic advanced table example
zamoore f044514
set up overflowing table for sortable demo
zamoore b6ec224
scrollIntoView works when reordering
zamoore 809e9c3
registering context menu toggle with the column class
zamoore e3715a6
refocusing the dropdown toggle when using the context menu to reorder
zamoore 76bba42
removed the console log
zamoore 7d2967a
responding to PR feedback
zamoore 5adee74
fixed an issue with reordering on sortable columns
zamoore 6b2ecde
Merge branch 'main' into zamoore/AdvancedTable/column-reordering-2
zamoore f7c9be1
added a changeset
zamoore 04c1ca1
updated changeset
zamoore bf3817b
Apply suggestions from code review
zamoore 51315d4
Merge branch 'main' into zamoore/AdvancedTable/column-reordering-2
zamoore 3fe499a
change where the top scroll indicator renders
zamoore 8e5cf3f
fixed a linting error
zamoore File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
--- | ||
"@hashicorp/design-system-components": minor | ||
--- | ||
|
||
<!-- START components/table/advanced-table --> | ||
`AdvancedTable` - Added support for column reordering. | ||
- Added `@hasReorderableColumns` argument. When set to `true`, enables column reordering. | ||
- Added optional `@columnOrder` argument for setting the initial order of columns by their keys. | ||
- Added optional `@onColumnReorder` argument which accepts a callback function that is called when reordering is completed. | ||
- Added optional `@reorderedMessageText` which overrides the default message text that is rendered in the table caption when a column is reordered. | ||
<!-- END --> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.