Blender Git Commit Log

Git Commits -> Revision e61588c

Revision e61588c by Julian Eisel (master)
February 29, 2016, 15:18 (GMT)
Fix a bunch of temp full-screen glitches

Steps to reproduce fixed glitches were:
* Change any editor to be file browser from menu, Ctrl+O *from the file browser area*, Esc -> area reset to what it was before changing to file browser initially
* Ctrl+O from any area, F12, Esc -> returns to initial editor in full-screen (expected is file browser in full-screen)

Fixes T46229

Core of the fix is removing old area from spacedata list when going back to previous area (see ED_area_prevspace -> BKE_spacedata_remove). Also, when creating a new temp area we now don't exit old area anymore (needed so SpaceFile->op is kept, but it also makes sense in general)
Aaand finally removes some ugly hacks.

Tested quite a bit, so I think it's safe to apply (besides of remark below), just would like to get things double checked and confirmed. After all, this full-screen stuff finally starts to feel like it's working :P

Note, there's still a memory leak when quitting Blender with temp area open. Haven't found out how to solve yet, but it's not that important for review anyway.

Reviewers: campbellbarton, brecht

Reviewed By: brecht

Subscribers: plyczkowski, Blendify

Maniphest Tasks: T46229

Differential Revision: https://developer.blender.org/D1531

Commit Details:

Full Hash: e61588c5a5446fe8b0d7c974c7619268d5900031
Parent Commit: 49ba974
Lines Changed: +46, -39

9 Modified Paths:

/source/blender/editors/include/ED_screen.h (+2, -2) (Diff)
/source/blender/editors/render/render_view.c (+5, -11) (Diff)
/source/blender/editors/screen/area.c (+23, -7) (Diff)
/source/blender/editors/screen/screen_edit.c (+5, -9) (Diff)
/source/blender/editors/screen/screen_ops.c (+1, -1) (Diff)
/source/blender/makesrna/intern/rna_screen.c (+1, -1) (Diff)
/source/blender/windowmanager/intern/wm_event_system.c (+6, -5) (Diff)
/source/blender/windowmanager/intern/wm_window.c (+2, -2) (Diff)
/source/blenderplayer/bad_level_call_stubs/stubs.c (+1, -1) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021