-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat: (React Aria) Implement filtering on a per CollectionNode basis #8641
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
Open
LFDanLu
wants to merge
32
commits into
main
Choose a base branch
from
baseCollection_filter
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
c9ee11d
account for loaders in base collection filter
LFDanLu ac323e9
rough implementation for listbox
LFDanLu 290514a
replace other instances of createLeaf/createBranch to use node classes
LFDanLu 2969511
fix bugs with subdialog filtering, arrow nav, dividers, etc
LFDanLu d8a6f06
fix case where arrow nav wasnt working post filter
LFDanLu 639acd0
Merge branch 'main' of github.com:adobe/react-spectrum into baseColle…
LFDanLu 6eb1753
update types and class node structure
LFDanLu d1efa8d
prep stories
LFDanLu 77936ca
fix
LFDanLu 7991977
add autocomplete gridlist filtering
LFDanLu e615835
taglist filter support
LFDanLu d02197e
fixing lint
LFDanLu 361286b
fix tag group keyboard nav and lint
LFDanLu 432a43c
adding support for table filtering
LFDanLu 3ec3fd6
fix tableCollection filter so it doesnt need to call filterChildren d…
LFDanLu 4a69d50
create common use nodes for specific filtering patterns
LFDanLu 73a1971
fix ssr
LFDanLu 1ead59b
refactor to accept a node rather than a string in the filter function
LFDanLu 90c2056
fix lint
LFDanLu 3a8301e
make node param in autocomplete non breaking
LFDanLu 45a39c1
Merge branch 'main' of github.com:adobe/react-spectrum into baseColle…
LFDanLu 9d65d5b
adding tests, make sure we only apply autocomplete attributes if the …
LFDanLu 19b695e
prevent breaking change in CollectionBuilder by still accepting strin…
LFDanLu 6066c6c
fix tests and pass submenutrigger node to filterFn
LFDanLu d2b5e51
small clean up
LFDanLu 2c89783
small fixes
LFDanLu 739e93f
addressing more review comments
LFDanLu 3c2e92c
simplifying setProps logic since we have already have id when calling it
LFDanLu 35b627e
forgot to use generic for autocomplete filter
LFDanLu 9408aa9
ugh docs typescript
LFDanLu 8e75339
review comments
LFDanLu 57e57e0
add example testing the Autocomplete generic
LFDanLu 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
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
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
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.
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.
For discussion, might be enough to just provide a subset of node information as mentioned above
Uh oh!
There was an error while loading. Please reload this page.
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.
Hm, would also ask to hold off with this until the RFC. While it is definitely enough to sync collections, I do fancy the idea of being able to attach a node as context on another node - doing it all in one iteration would be great.
PS: I guess key in the end is always enough though since one can just retrieve the node.
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.
happy to hold off on paring it down for now, but on the flip side it is easier to go from exposing object containing a subset of the Node's values back to a Node if the need arises.