Proper integration of File-selecting in WM. The communication
flow was flawed. :) Main problem was that filewindow can change
the screen context entirely, and should not do this directly on
a call inside an operator. Another problem was that the operator
ownership was handed over to SpaceFile, which is asking for
problems if you want to execute the operator with proper context
later on.
Solution is simple; window handlers already are valid owners of
operators and can manage context, so instead of directly talking
to the 'file space', you give the operator to a new handler this
way:
WM_event_add_fileselect(C, op);
This handler then listens to events (OPEN, EXEC, CANCEL) sent
by the WM or by the filewindow itself. This way local context
operators (like "open new image in imagewindow") will survive
a full-window fileselector fine, and in future also secondary
windows browsing files.
Two bugfixes included in this commit too:
- Add sequence menus in Sequencer used wrong context.
- When handler executes operators, it sets stored context now
by first checking if this is still valid.
97 lines
2.7 KiB
C++
97 lines
2.7 KiB
C++
/**
|
|
* $Id:
|
|
*
|
|
* ***** BEGIN GPL LICENSE BLOCK *****
|
|
*
|
|
* This program is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU General Public License
|
|
* as published by the Free Software Foundation; either version 2
|
|
* of the License, or (at your option) any later version.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program; if not, write to the Free Software Foundation,
|
|
* Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
|
|
*
|
|
* The Original Code is Copyright (C) 2007 Blender Foundation.
|
|
* All rights reserved.
|
|
*
|
|
*
|
|
* Contributor(s): Blender Foundation
|
|
*
|
|
* ***** END GPL LICENSE BLOCK *****
|
|
*/
|
|
#ifndef WM_EVENT_SYSTEM_H
|
|
#define WM_EVENT_SYSTEM_H
|
|
|
|
/* return value of handler-operator call */
|
|
#define WM_HANDLER_CONTINUE 0
|
|
#define WM_HANDLER_BREAK 1
|
|
|
|
struct ScrArea;
|
|
struct ARegion;
|
|
|
|
/* wmKeyMap is in DNA_windowmanager.h, it's savable */
|
|
|
|
typedef struct wmEventHandler {
|
|
struct wmEventHandler *next, *prev;
|
|
|
|
int type, flag; /* type default=0, rest is custom */
|
|
|
|
/* keymap handler */
|
|
ListBase *keymap; /* pointer to builtin/custom keymaps */
|
|
|
|
rcti *bblocal, *bbwin; /* optional local and windowspace bb */
|
|
|
|
/* modal operator handler */
|
|
wmOperator *op; /* for derived/modal handlers */
|
|
struct ScrArea *op_area; /* for derived/modal handlers */
|
|
struct ARegion *op_region; /* for derived/modal handlers */
|
|
|
|
/* ui handler */
|
|
wmUIHandlerFunc ui_handle; /* callback receiving events */
|
|
wmUIHandlerRemoveFunc ui_remove; /* callback when handler is removed */
|
|
void *ui_userdata; /* user data pointer */
|
|
struct ScrArea *ui_area; /* for derived/modal handlers */
|
|
struct ARegion *ui_region; /* for derived/modal handlers */
|
|
|
|
/* fileselect handler re-uses modal operator data */
|
|
struct bScreen *filescreen; /* screen it started in, to validate exec */
|
|
|
|
} wmEventHandler;
|
|
|
|
|
|
/* handler flag */
|
|
/* after this handler all others are ignored */
|
|
#define WM_HANDLER_BLOCKING 1
|
|
|
|
|
|
|
|
/* custom types for handlers, for signalling, freeing */
|
|
enum {
|
|
WM_HANDLER_DEFAULT,
|
|
WM_HANDLER_FILESELECT
|
|
};
|
|
|
|
|
|
/* wm_event_system.c */
|
|
void wm_event_add (wmWindow *win, wmEvent *event_to_add);
|
|
void wm_event_free_all (wmWindow *win);
|
|
|
|
/* goes over entire hierarchy: events -> window -> screen -> area -> region */
|
|
void wm_event_do_handlers (bContext *C);
|
|
|
|
void wm_event_add_ghostevent (wmWindow *win, int type, void *customdata);
|
|
|
|
void wm_event_do_notifiers (bContext *C);
|
|
|
|
/* wm_keymap.c */
|
|
|
|
|
|
#endif /* WM_EVENT_SYSTEM_H */
|
|
|