Skip to content

[DC-231] improvements for tree navigation and edit states#12468

Merged
modSpike merged 21 commits intomasterfrom
refactor/dc-231
Mar 10, 2026
Merged

[DC-231] improvements for tree navigation and edit states#12468
modSpike merged 21 commits intomasterfrom
refactor/dc-231

Conversation

@modSpike
Copy link
Contributor

@modSpike modSpike commented Mar 6, 2026

modSpike added 4 commits March 6, 2026 14:49
redundant: small cleanup
added an event filter to allow setting the current item to edit mode when tree view gets focus.

also updated the paints for the folder item delegate to better deal with the tree's indent.

updated some comments too
added a "multiload" switch to prevent onFolderAdded from sorting the model on every call.

also refined the behavior on current item change
redundant: as it says in the description
@update-docs
Copy link

update-docs bot commented Mar 6, 2026

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@modSpike modSpike marked this pull request as draft March 6, 2026 14:22
modSpike added 17 commits March 6, 2026 15:22
also made sure all "actionable" widgets have strong focus
copy pasta ick, the return type on messagebox exec did not match the handling conditions
small bug fix that make sure current selected item in tree is scrolled to when focus in (via keyboard, obvs)
the folder wizard was fine when run from the "add folder" button but completely busted when run via the account wizard/selective sync option.

this is why duplicated code is BAD. I will refactor more heavily to end up with a single impl for this, but for now, need to get squish updated so this simpler fix is ok.
@modSpike modSpike marked this pull request as ready for review March 10, 2026 10:50
@modSpike modSpike self-assigned this Mar 10, 2026
@modSpike modSpike requested a review from DeepDiver1975 March 10, 2026 10:50
@modSpike modSpike enabled auto-merge (squash) March 10, 2026 10:52
@modSpike modSpike merged commit a5229ba into master Mar 10, 2026
8 checks passed
@modSpike modSpike deleted the refactor/dc-231 branch March 10, 2026 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants