@JulienKaspar - Who from the UI module should be included in the review process?
I added @JulianEisel as reviewer
Are there other modules that would have a stake in this?
I don't…
@ideasman42 - Can you think of any downsides to this?
Taking a quick look I see nothing obviously wrong so I might be a good time to remove "WIP", assuming it does everything it intends to. But I'd guess the review could be a longer process. This is…
@HooglyBoogly - If tree isn't null, tree.catalogs will never be null
At the time of the crash the tree existed in that I could examine its contents, but for its "catalogs" my MSVS debugger…
Not able to reproduce it here.
Tested with 3.5. Installed "3D viewport pie menus" addon. Alt-Q to bring up the "View Numpad" pie. It looks the same no matter what input language is selected:
…
Following these instructions exactly I see this same error with an exception in uiTemplateNodeAssetMenuItems while iterating through an AssetCatalogTree with find_root_item.
A…
First I should say that I am entirely and enthusiastically in favor of all content keeping the same vertical direction regardless of where it is spawned. I have proposed this many times in the…
Seems reasonable. Although I'd probably use BLI_listbase_is_empty(&wm->drags) though.
@ideasman42 - I'd rather avoid exposing internal implementation details and instead have...
Yes, it is much nicer like that. Added BLI_str_cursor_at_word_boundary_utfx() to clean that up and…