Remove redundant file bookmarks region toggle operator
For the default keymap we were only using the regular toolshelf operator, doing this for the industry compatible keymap too now (we could even remove it there, we don't use it in other editors). Since we "now" have proper operators for toggling regions, this specific one is totally redundant.
This commit is contained in:
@@ -82,7 +82,6 @@ void FILE_OT_directory_new(struct wmOperatorType *ot);
|
||||
void FILE_OT_previous(struct wmOperatorType *ot);
|
||||
void FILE_OT_next(struct wmOperatorType *ot);
|
||||
void FILE_OT_refresh(struct wmOperatorType *ot);
|
||||
void FILE_OT_bookmark_toggle(struct wmOperatorType *ot);
|
||||
void FILE_OT_filenum(struct wmOperatorType *ot);
|
||||
void FILE_OT_delete(struct wmOperatorType *ot);
|
||||
void FILE_OT_rename(struct wmOperatorType *ot);
|
||||
|
||||
@@ -2315,30 +2315,6 @@ void FILE_OT_hidedot(struct wmOperatorType *ot)
|
||||
ot->poll = ED_operator_file_active; /* <- important, handler is on window level */
|
||||
}
|
||||
|
||||
static int file_bookmark_toggle_exec(bContext *C, wmOperator *UNUSED(unused))
|
||||
{
|
||||
ScrArea *sa = CTX_wm_area(C);
|
||||
ARegion *ar = BKE_area_find_region_type(sa, RGN_TYPE_TOOLS);
|
||||
|
||||
if (ar) {
|
||||
ED_region_toggle_hidden(C, ar);
|
||||
}
|
||||
|
||||
return OPERATOR_FINISHED;
|
||||
}
|
||||
|
||||
void FILE_OT_bookmark_toggle(struct wmOperatorType *ot)
|
||||
{
|
||||
/* identifiers */
|
||||
ot->name = "Toggle Bookmarks";
|
||||
ot->description = "Toggle bookmarks display";
|
||||
ot->idname = "FILE_OT_bookmark_toggle";
|
||||
|
||||
/* api callbacks */
|
||||
ot->exec = file_bookmark_toggle_exec;
|
||||
ot->poll = ED_operator_file_active; /* <- important, handler is on window level */
|
||||
}
|
||||
|
||||
static bool file_filenum_poll(bContext *C)
|
||||
{
|
||||
SpaceFile *sfile = CTX_wm_space_file(C);
|
||||
|
||||
@@ -562,7 +562,6 @@ static void file_operatortypes(void)
|
||||
WM_operatortype_append(FILE_OT_previous);
|
||||
WM_operatortype_append(FILE_OT_next);
|
||||
WM_operatortype_append(FILE_OT_refresh);
|
||||
WM_operatortype_append(FILE_OT_bookmark_toggle);
|
||||
WM_operatortype_append(FILE_OT_bookmark_add);
|
||||
WM_operatortype_append(FILE_OT_bookmark_delete);
|
||||
WM_operatortype_append(FILE_OT_bookmark_cleanup);
|
||||
|
||||
Reference in New Issue
Block a user