UI: Add search-create callback
authorCampbell Barton <ideasman42@gmail.com>
Wed, 2 Mar 2016 02:57:16 +0000 (13:57 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 2 Mar 2016 02:57:16 +0000 (13:57 +1100)
Needed to support alternate search UI's, no functional changes.

source/blender/editors/include/UI_interface.h
source/blender/editors/interface/interface.c
source/blender/editors/interface/interface_handlers.c
source/blender/editors/interface/interface_intern.h
source/blender/editors/interface/interface_layout.c
source/blender/editors/interface/interface_regions.c
source/blender/editors/interface/interface_templates.c
source/blender/editors/space_node/node_select.c
source/blender/editors/space_view3d/view3d_toolbar.c
source/blender/windowmanager/intern/wm_operators.c

index 465ab4e1d91593698dfc4dfca7ab1742b5c44ad5..0f0e40850cc67a189d819341fec3e93c7b4218b7 100644 (file)
@@ -354,6 +354,7 @@ typedef void (*uiButHandleFunc)(struct bContext *C, void *arg1, void *arg2);
 typedef void (*uiButHandleRenameFunc)(struct bContext *C, void *arg, char *origstr);
 typedef void (*uiButHandleNFunc)(struct bContext *C, void *argN, void *arg2);
 typedef int (*uiButCompleteFunc)(struct bContext *C, char *str, void *arg);
+typedef struct ARegion *(*uiButSearchCreateFunc)(struct bContext *C, struct ARegion *butregion, uiBut *but);
 typedef void (*uiButSearchFunc)(const struct bContext *C, void *arg, const char *str, uiSearchItems *items);
 /* Must return allocated string. */
 typedef char *(*uiButToolTipFunc)(struct bContext *C, void *argN, const char *tip);
@@ -680,7 +681,9 @@ uiBut *UI_block_links_find_inlink(uiBlock *block, void *poin);
 /* use inside searchfunc to add items */
 bool    UI_search_item_add(uiSearchItems *items, const char *name, void *poin, int iconid);
 /* bfunc gets search item *poin as arg2, or if NULL the old string */
-void    UI_but_func_search_set(uiBut *but,        uiButSearchFunc sfunc, void *arg1, uiButHandleFunc bfunc, void *active);
+void    UI_but_func_search_set(
+        uiBut *but, uiButSearchCreateFunc cfunc, uiButSearchFunc sfunc,
+        void *arg1, uiButHandleFunc bfunc, void *active);
 /* height in pixels, it's using hardcoded values still */
 int     UI_searchbox_size_y(void);
 int     UI_searchbox_size_x(void);
index bca37947acea3baf37131737bbac63d30c6542d6..32c9177fafe837cbba3a17c09034c871d29fb0fe 100644 (file)
@@ -4320,9 +4320,19 @@ uiBut *uiDefSearchBut(uiBlock *block, void *arg, int retval, int icon, int maxle
  * \param arg: user value,
  * \param  active: when set, button opens with this item visible and selected.
  */
-void UI_but_func_search_set(uiBut *but, uiButSearchFunc sfunc, void *arg, uiButHandleFunc bfunc, void *active)
+void UI_but_func_search_set(
+        uiBut *but,
+        uiButSearchCreateFunc search_create_func,
+        uiButSearchFunc search_func, void *arg,
+        uiButHandleFunc bfunc, void *active)
 {
-       but->search_func = sfunc;
+       /* needed since callers don't have access to internal functions (as an alternative we could expose it) */
+       if (search_create_func == NULL) {
+               search_create_func = ui_searchbox_create_generic;
+       }
+
+       but->search_create_func = search_create_func;
+       but->search_func = search_func;
        but->search_arg = arg;
        
        UI_but_func_set(but, bfunc, arg, active);
@@ -4400,7 +4410,9 @@ uiBut *uiDefSearchButO_ptr(
        uiBut *but;
 
        but = uiDefSearchBut(block, arg, retval, icon, maxlen, x, y, width, height, a1, a2, tip);
-       UI_but_func_search_set(but, operator_enum_search_cb, but, operator_enum_call_cb, NULL);
+       UI_but_func_search_set(
+               but, ui_searchbox_create_generic, operator_enum_search_cb,
+               but, operator_enum_call_cb, NULL);
 
        but->optype = ot;
        but->opcontext = WM_OP_EXEC_DEFAULT;
index 0ae505777a93a55a46ad8154613649374ecfb0d0..2c99c9be657f6bf2fc4eb65643075f0c44694629 100644 (file)
@@ -3066,7 +3066,7 @@ static void ui_textedit_begin(bContext *C, uiBut *but, uiHandleButtonData *data)
 
        /* optional searchbox */
        if (but->type ==  UI_BTYPE_SEARCH_MENU) {
-               data->searchbox = ui_searchbox_create(C, data->region, but);
+               data->searchbox = but->search_create_func(C, data->region, but);
                ui_searchbox_update(C, data->searchbox, but, true); /* true = reset */
        }
 
index 1fced5165389aaceaffc9f1967b23028f91645c8..99656b89ee219954ba83daebca12a6a8c6718cdc 100644 (file)
@@ -255,6 +255,7 @@ struct uiBut {
        uiButCompleteFunc autocomplete_func;
        void *autofunc_arg;
        
+       uiButSearchCreateFunc search_create_func;
        uiButSearchFunc search_func;
        void *search_arg;
 
@@ -591,7 +592,7 @@ void ui_color_picker_to_rgb_v(const float r_cp[3], float rgb[3]);
 void ui_color_picker_to_rgb(float r_cp0, float r_cp1, float r_cp2, float *r, float *g, float *b);
 
 /* searchbox for string button */
-ARegion *ui_searchbox_create(struct bContext *C, struct ARegion *butregion, uiBut *but);
+ARegion *ui_searchbox_create_generic(struct bContext *C, struct ARegion *butregion, uiBut *but);
 bool ui_searchbox_inside(struct ARegion *ar, int x, int y);
 int  ui_searchbox_find_index(struct ARegion *ar, const char *name);
 void ui_searchbox_update(struct bContext *C, struct ARegion *ar, uiBut *but, const bool reset);
index 4d9c1b387a19557990fad55263aef7c9b08f2db2..2daf3f7ba70c92c7501bc331f738e9b1fa0f9436 100644 (file)
@@ -1672,7 +1672,7 @@ void ui_but_add_search(uiBut *but, PointerRNA *ptr, PropertyRNA *prop, PointerRN
                        but->str[0] = 0;
                }
 
-               UI_but_func_search_set(but, rna_search_cb, but, NULL, NULL);
+               UI_but_func_search_set(but, ui_searchbox_create_generic, rna_search_cb, but, NULL, NULL);
        }
 }
 
index 8ee31c116464f11ae9449740b39ef95331c4ce9d..2a309fc126ccc27a7b914a3c04c086e05a9862ff 100644 (file)
@@ -829,7 +829,7 @@ int UI_searchbox_size_y(void)
 
 int UI_searchbox_size_x(void)
 {
-       return 12 * UI_UNIT_X;
+       return 10 * UI_UNIT_X;
 }
 
 int UI_search_items_find_index(uiSearchItems *items, const char *name)
@@ -1164,7 +1164,7 @@ static void ui_searchbox_region_free_cb(ARegion *ar)
        ar->regiondata = NULL;
 }
 
-ARegion *ui_searchbox_create(bContext *C, ARegion *butregion, uiBut *but)
+ARegion *ui_searchbox_create_generic(bContext *C, ARegion *butregion, uiBut *but)
 {
        wmWindow *win = CTX_wm_window(C);
        uiStyle *style = UI_style_get();
index e3a749d34f8a6f5635b11abf1c74ed3d11bb6a51..079205b4f0d2563a1063bdb0dc8db9c904f7fe33 100644 (file)
@@ -197,7 +197,9 @@ static uiBlock *id_search_menu(bContext *C, ARegion *ar, void *arg_litem)
                
                but = uiDefSearchBut(block, search, 0, ICON_VIEWZOOM, sizeof(search), 10, 0, w, UI_UNIT_Y,
                                     template.prv_rows, template.prv_cols, "");
-               UI_but_func_search_set(but, id_search_cb, &template, id_search_call_cb, idptr.data);
+               UI_but_func_search_set(
+                       but, ui_searchbox_create_generic, id_search_cb,
+                       &template, id_search_call_cb, idptr.data);
        }
        /* list view */
        else {
@@ -207,7 +209,9 @@ static uiBlock *id_search_menu(bContext *C, ARegion *ar, void *arg_litem)
                /* fake button, it holds space for search items */
                uiDefBut(block, UI_BTYPE_LABEL, 0, "", 10, 15, searchbox_width, searchbox_height, NULL, 0, 0, 0, 0, NULL);
                but = uiDefSearchBut(block, search, 0, ICON_VIEWZOOM, sizeof(search), 10, 0, searchbox_width, UI_UNIT_Y - 1, 0, 0, "");
-               UI_but_func_search_set(but, id_search_cb, &template, id_search_call_cb, idptr.data);
+               UI_but_func_search_set(
+                       but, ui_searchbox_create_generic, id_search_cb,
+                       &template, id_search_call_cb, idptr.data);
        }
                
        
@@ -3303,7 +3307,9 @@ static void operator_search_cb(const bContext *C, void *UNUSED(arg), const char
 
 void UI_but_func_operator_search(uiBut *but)
 {
-       UI_but_func_search_set(but, operator_search_cb, NULL, operator_call_cb, NULL);
+       UI_but_func_search_set(
+               but, ui_searchbox_create_generic, operator_search_cb,
+               NULL, operator_call_cb, NULL);
 }
 
 void uiTemplateOperatorSearch(uiLayout *layout)
index e2968b7003c01de91c57bd865991d16c8c3b77a8..7195d2a9838777571d076fa61d0e624223a14e9c 100644 (file)
@@ -1018,7 +1018,7 @@ static uiBlock *node_find_menu(bContext *C, ARegion *ar, void *arg_op)
        UI_block_flag_enable(block, UI_BLOCK_LOOP | UI_BLOCK_MOVEMOUSE_QUIT | UI_BLOCK_SEARCH_MENU);
        
        but = uiDefSearchBut(block, search, 0, ICON_VIEWZOOM, sizeof(search), 10, 10, 9 * UI_UNIT_X, UI_UNIT_Y, 0, 0, "");
-       UI_but_func_search_set(but, node_find_cb, op->type, node_find_call_cb, NULL);
+       UI_but_func_search_set(but, NULL, node_find_cb, op->type, node_find_call_cb, NULL);
        
        /* fake button, it holds space for search items */
        uiDefBut(block, UI_BTYPE_LABEL, 0, "", 10, 10 - UI_searchbox_size_y(), UI_searchbox_size_x(), UI_searchbox_size_y(), NULL, 0, 0, 0, 0, NULL);
index 065c336d00153413ce13d1c6707bb87948a5b444..62f605ab99c40097b9dd246ec4f2a8d3bac873bf 100644 (file)
@@ -185,7 +185,7 @@ static uiBlock *tool_search_menu(bContext *C, ARegion *ar, void *arg_listbase)
        uiDefBut(block, UI_BTYPE_LABEL, 0, "", 10, 15, UI_searchbox_size_x(), UI_searchbox_size_y(), NULL, 0, 0, 0, 0, NULL);
        
        but = uiDefSearchBut(block, search, 0, ICON_VIEWZOOM, sizeof(search), 10, 0, 150, 19, 0, 0, "");
-       UI_but_func_search_set(but, operator_search_cb, arg_listbase, operator_call_cb, NULL);
+       UI_but_func_search_set(but, NULL, operator_search_cb, arg_listbase, operator_call_cb, NULL);
        
        UI_block_bounds_set_normal(block, 6);
        UI_block_direction_set(block, UI_DIR_DOWN);
index 8a9b069bdc3f9f34dedd6f35845c0e478a3868e4..16f12d41b068eca5a7ba39fc65ce0b73cbf4275b 100644 (file)
@@ -1900,8 +1900,14 @@ static void WM_OT_splash(wmOperatorType *ot)
 
 
 /* ***************** Search menu ************************* */
-static uiBlock *wm_block_search_menu(bContext *C, ARegion *ar, void *UNUSED(arg_op))
+
+struct SearchPopupInit_Data {
+       int size[2];
+};
+
+static uiBlock *wm_block_search_menu(bContext *C, ARegion *ar, void *userdata)
 {
+       const struct SearchPopupInit_Data *init_data = userdata;
        static char search[256] = "";
        wmEvent event;
        wmWindow *win = CTX_wm_window(C);
@@ -1911,11 +1917,12 @@ static uiBlock *wm_block_search_menu(bContext *C, ARegion *ar, void *UNUSED(arg_
        block = UI_block_begin(C, ar, "_popup", UI_EMBOSS);
        UI_block_flag_enable(block, UI_BLOCK_LOOP | UI_BLOCK_MOVEMOUSE_QUIT | UI_BLOCK_SEARCH_MENU);
        
-       but = uiDefSearchBut(block, search, 0, ICON_VIEWZOOM, sizeof(search), 10, 10, UI_searchbox_size_x(), UI_UNIT_Y, 0, 0, "");
+       but = uiDefSearchBut(block, search, 0, ICON_VIEWZOOM, sizeof(search), 10, 10, init_data->size[0], UI_UNIT_Y, 0, 0, "");
        UI_but_func_operator_search(but);
        
        /* fake button, it holds space for search items */
-       uiDefBut(block, UI_BTYPE_LABEL, 0, "", 10, 10 - UI_searchbox_size_y(), UI_searchbox_size_x(), UI_searchbox_size_y(), NULL, 0, 0, 0, 0, NULL);
+       uiDefBut(block, UI_BTYPE_LABEL, 0, "", 10, 10 - init_data->size[1],
+                init_data->size[0], init_data->size[1], NULL, 0, 0, 0, 0, NULL);
        
        UI_block_bounds_set_popup(block, 6, 0, -UI_UNIT_Y); /* move it downwards, mouse over button */
        
@@ -1934,9 +1941,16 @@ static int wm_search_menu_exec(bContext *UNUSED(C), wmOperator *UNUSED(op))
        return OPERATOR_FINISHED;
 }
 
-static int wm_search_menu_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSED(event))
+static int wm_search_menu_invoke(bContext *C, wmOperator *UNUSED(op), const wmEvent *UNUSED(event))
 {
-       UI_popup_block_invoke(C, wm_block_search_menu, op);
+       struct SearchPopupInit_Data data = {
+               .size = {
+                   UI_searchbox_size_x(),
+                   UI_searchbox_size_y(),
+               },
+       };
+
+       UI_popup_block_invoke(C, wm_block_search_menu, &data);
        
        return OPERATOR_INTERFACE;
 }