Skip to content
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

Implement Sorting and Filtering of Changes based on Branches #215

Merged
merged 3 commits into from
Jul 26, 2024

Conversation

F12-Syntex
Copy link

@F12-Syntex F12-Syntex commented Jul 24, 2024

This PR implements #211.

It provides a list of branches that can be sorted by:

  • most recent
  • most used
  • alphabetic

Selecting a branch filters the changes.
The list of branches itself can be filter by a search field, too.

- shows a list of branches on the left and right side
- allows to search for branches
- allows to sort branches by most used, most recent, or alphabetically

Signed-off-by: Stefan Marr <git@stefan-marr.de>
- adapt naming to match the implementation
  - sort vs filter, cards-container vs list

- added added type to get better tool feedback
- refactor code to:
  - only request data once from the server
  - process branch details only once
  - eliminate redundant code to add changes to the changes lists (with and without filtering)
  - break things up into more functions
  - use .data() function to access the triggered element and avoid having to create new closures for each element

Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr changed the title Simplify filter system Implement Sorting and Filter of Changes based on Branches Jul 25, 2024
@smarr smarr changed the title Implement Sorting and Filter of Changes based on Branches Implement Sorting and Filtering of Changes based on Branches Jul 25, 2024
@smarr smarr added the enhancement New feature or request label Jul 25, 2024
Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr merged commit 792af37 into smarr:master Jul 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants