Fix T67008: Missing move handle and flickering in FileBrowser
Allow split regions (child regions) to contribute to the action zones (azone) of the parent region.
This fixes the issues in file browser and also in the user preferences.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- local_temp_T67008 (branched from master)
- Build Status
Buildable 4087 Build 4087: arc lint + arc unit
Event Timeline
Comment Actions
Thanks for the patch! It seems to work nicely, and I cannot find anything wrong in its code. But am really not familiar with that area (haha) of the UI code, so would not mind if @Campbell Barton (campbellbarton) or @Brecht Van Lommel (brecht) could also have a look at it?
Comment Actions
I'm probably the one who did most changes for 2.80 in relating code. And I didn't remember ever seeing special treatment of split regions, which made me wonder how this could ever work before.
Turns out we added the AZone's during region size updating, at a point where the prev region was temporarily using a winrct of both regions combined. So of course a single AZone spanning both regions would be added.
Given that we've now separated region resizing from AZone creation, this kind of special treatment makes sense. Didn't test, but the code seems to handle everything needed.