-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
FileStatusList: Add toolbar and actions on folders #12149
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
FileStatusList: Add toolbar and actions on folders #12149
Conversation
A few comments on usage:
Noted, probably OK:
Some enhancements, probably future PRs:
|
Yes, not nice. It disappears on mouse move. Closing the dropdown does not work either. Blame WF.
If there are modified
I do not like to exchange menu item texts on runtime unless really necessary. I think opening the repo folder for root / group-by nodes is not counter-intuitive.
If you mean "Show in folder": A folder is selected. So I think, that this folder shall be selected in the Explorer.
File tree needs slightly different handling. E.g. grouping does not really apply. I prefer to not add more groups, especially no expensive ones. It would need to be excluded from "find text in files".
👍 for those who do not use auto-refresh I am going to add a split button with this setting, too. GitStatusMonitor status to be added in a follow-up |
Remove from PR?
It was not what I expected. No objection for now.
OK
This group should only be calculated when expanded, we leave it here for now.
Auto refresh is too slow for me in some repos
I would have preferred to have it leftmost as for the main toolbar and being disabled if no artificial. Now it is moving places... |
It would have to compare current status with the update, if ever implemented.
Agree, numbers are in the group (may be different) and grid. |
d0ba2e6
to
cebdb3a
Compare
cebdb3a
to
462593f
Compare
462593f
to
216c28c
Compare
on single click Refs: gitextensions#12149
if it is a single item (i.e. a leaf node) Refs: gitextensions#12149
66586cd
to
ff95d37
Compare
if it is a single item (i.e. a leaf node) Refs: gitextensions#12149
ff95d37
to
3762e9d
Compare
Sorry, yet another two addenda |
if it is a single item (i.e. a leaf node) Refs: gitextensions#12149
3762e9d
to
43dbda3
Compare
Huge effort, congratulations! 🚀 |
Proposed changes
RevisionDiffControl
if folder selected:FileStatusList
:Screenshots
Before
After
Test methodology
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.