-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
BranchFilter text was added backwards #10136
BranchFilter text was added backwards #10136
Conversation
Dropdown was not opened on focus, which made the filter unusable. Cache the function for getting the references with lazy result, to speed up the dropdown.
Could you please provide an animation/video of the new behaviour? |
Not much different from now and less weird. |
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.
Thank you
Will merge this in a batch with other PRs that should be merged and update release/4.0 too. The implementation just restores 3.5 handling (and some performance improvements), a proper solution should be investigated. |
👍
|
Dropdown was not opened on focus, which made the filter unusable. Cache the function for getting the references with lazy result, to speed up the dropdown. (cherry picked from commit bb9b967)
The branchfilters from the sidepanel was not applied unless the filter dropdown contained any items, but the items were not updated by default since gitextensions#10136, only when needed (dropdown clicked or text typed). The dropdown was updated async FileAndForget, so 'external' usage was not always successful. This was a reason why the test 'Filters_should_behave_as_expected' had to be disabled in AppVeyor (the test still occasionally fails though).
The branchfilters from the sidepanel was not applied unless the filter dropdown contained any items, but the items were not updated by default since gitextensions#10136, only when needed (dropdown clicked or text typed). The dropdown was updated async FileAndForget, so 'external' usage was not always successful. This was a reason why the test 'Filters_should_behave_as_expected' had to be disabled in AppVeyor (the test still occasionally fails though).
The branchfilters from the sidepanel was not applied unless the filter dropdown contained any items, but the items were not updated by default since #10136, only when needed (dropdown clicked or text typed). The dropdown was updated async FileAndForget, so 'external' usage was not always successful. This was a reason why the test 'Filters_should_behave_as_expected' had to be disabled in AppVeyor (the test still occasionally fails though).
The branchfilters from the sidepanel was not applied unless the filter dropdown contained any items, but the items were not updated by default since #10136, only when needed (dropdown clicked or text typed). The dropdown was updated async FileAndForget, so 'external' usage was not always successful. This was a reason why the test 'Filters_should_behave_as_expected' had to be disabled in AppVeyor (the test still occasionally fails though). (cherry picked from commit caaccf5)
Fixes #9872
Proposed changes
Clearing the dropdown items changed the focus and moved the focus in the textbox to the first position, so next key was added first.
Dropdown was not opened on focus, which made the filter unusable unless you opened the dropdown first (se below).
Cache the function for getting the references with lazy result, to speed up the dropdown.
Still some weird behavior:
This PR is not perfect but it is similar to the 3.5 behavior and I have no WinForms knowledge and I would like to keep it at this.
Test methodology
manual
Merge strategy
I agree that the maintainer squash merge this PR (if the commit message is clear).
✒️ I contribute this code under The Developer Certificate of Origin.