Fix T89526: "Toggle Maximize Area" clears context screen properties
Removed in b787581c9c
as it's comment
noted it was bad code, the reason for it's necessity was no longer valid.
Add this back with comment explaining why it's still needed.
This commit is contained in:
@@ -1564,6 +1564,14 @@ ScrArea *ED_screen_state_toggle(bContext *C, wmWindow *win, ScrArea *area, const
|
||||
BLI_assert(CTX_wm_screen(C) == screen);
|
||||
BLI_assert(CTX_wm_area(C) == NULL); /* May have been freed. */
|
||||
|
||||
/* Setting the area is only needed for Python scripts that call
|
||||
* operators in succession before returning to the main event loop.
|
||||
* Without this, scripts can't run any operators that require
|
||||
* an area after toggling full-screen for example (see: T89526).
|
||||
* NOTE: an old comment stated this was "bad code",
|
||||
* however it doesn't cause problems so leave as-is. */
|
||||
CTX_wm_area_set(C, screen->areabase.first);
|
||||
|
||||
return screen->areabase.first;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user