From 78d2ce19c4fbb5678995efae74a6128a3dcc4e73 Mon Sep 17 00:00:00 2001 From: Julian Eisel Date: Wed, 23 Dec 2020 13:56:57 +0100 Subject: Fix T84013: Crash closing maximized File Browser opened from Preferences After 1e799dd26ec1, the logic to recognize a temporary File Browser window didn't work correctly anymore. It would recognize a maximized File Browser inside a temporary window as a temporary File Browser window, and attempt to close this (rather than returning to the previous layout). The logic there was pretty weak, and still is I think. A more stable solution would need bigger refactoring. With this, it's also not possible to maximize or fullscreen an area within a temporary window (Preferences, File Browser, render window) anymore. Think that won't make a noticable difference, since you couldn't open multiple areas there anyway, and the area seems to be maximized already. Cleaned up the code a bit to not become more confusing with the changes. --- .../blender/windowmanager/intern/wm_event_system.c | 69 ++++++++++++---------- 1 file changed, 39 insertions(+), 30 deletions(-) (limited to 'source/blender/windowmanager') diff --git a/source/blender/windowmanager/intern/wm_event_system.c b/source/blender/windowmanager/intern/wm_event_system.c index 20fa500f0b3..90aa6a92d3c 100644 --- a/source/blender/windowmanager/intern/wm_event_system.c +++ b/source/blender/windowmanager/intern/wm_event_system.c @@ -2259,38 +2259,47 @@ static int wm_handler_fileselect_do(bContext *C, bScreen *screen = WM_window_get_active_screen(win); ScrArea *file_area = screen->areabase.first; - if (screen->temp && (file_area->spacetype == SPACE_FILE)) { - int win_size[2]; - bool is_maximized; - ED_fileselect_window_params_get(win, win_size, &is_maximized); - ED_fileselect_params_to_userdef(file_area->spacedata.first, win_size, is_maximized); - - if (BLI_listbase_is_single(&file_area->spacedata)) { - BLI_assert(ctx_win != win); - - wm_window_close(C, wm, win); - - CTX_wm_window_set(C, ctx_win); /* #wm_window_close() NULLs. */ - /* Some operators expect a drawable context (for EVT_FILESELECT_EXEC). */ - wm_window_make_drawable(wm, ctx_win); - /* Ensure correct cursor position, otherwise, popups may close immediately after - * opening (UI_BLOCK_MOVEMOUSE_QUIT). */ - wm_get_cursor_position(ctx_win, &ctx_win->eventstate->x, &ctx_win->eventstate->y); - wm->winactive = ctx_win; /* Reports use this... */ - if (handler->context.win == win) { - handler->context.win = NULL; - } - } - else if (file_area->full) { - ED_screen_full_prevspace(C, file_area); - } - else { - ED_area_prevspace(C, file_area); - } + if ((file_area->spacetype != SPACE_FILE) || !WM_window_is_temp_screen(win)) { + continue; + } - temp_win = win; - break; + if (ctx_area->full) { + /* Users should not be able to maximize/fullscreen an area in a temporary screen. So if + * there's a maximized file browser in a temporary screen, it was likely opened by + * #EVT_FILESELECT_FULL_OPEN. */ + continue; } + + int win_size[2]; + bool is_maximized; + ED_fileselect_window_params_get(win, win_size, &is_maximized); + ED_fileselect_params_to_userdef(file_area->spacedata.first, win_size, is_maximized); + + if (BLI_listbase_is_single(&file_area->spacedata)) { + BLI_assert(ctx_win != win); + + wm_window_close(C, wm, win); + + CTX_wm_window_set(C, ctx_win); /* #wm_window_close() NULLs. */ + /* Some operators expect a drawable context (for EVT_FILESELECT_EXEC). */ + wm_window_make_drawable(wm, ctx_win); + /* Ensure correct cursor position, otherwise, popups may close immediately after + * opening (UI_BLOCK_MOVEMOUSE_QUIT). */ + wm_get_cursor_position(ctx_win, &ctx_win->eventstate->x, &ctx_win->eventstate->y); + wm->winactive = ctx_win; /* Reports use this... */ + if (handler->context.win == win) { + handler->context.win = NULL; + } + } + else if (file_area->full) { + ED_screen_full_prevspace(C, file_area); + } + else { + ED_area_prevspace(C, file_area); + } + + temp_win = win; + break; } if (!temp_win && ctx_area->full) { -- cgit v1.2.3