rimmed pushed a commit to branch master. http://git.enlightenment.org/tools/eflete.git/commit/?id=7f7458453f622c62e63dca870d36438899fbe98f
commit 7f7458453f622c62e63dca870d36438899fbe98f Author: Vitalii Vorobiov <vi.vorob...@samsung.com> Date: Mon Mar 21 14:06:06 2016 +0200 shortcuts: cleanup unused shortcuts Only few are allowed: save, quit, close tab, undo, redo, zoom in, zoom out, show object area and change align --- src/bin/config/config.c | 36 +--- src/bin/ui/shortcuts/shortcuts.c | 414 +-------------------------------------- 2 files changed, 8 insertions(+), 442 deletions(-) diff --git a/src/bin/config/config.c b/src/bin/config/config.c index ecca51e..76b98be 100644 --- a/src/bin/config/config.c +++ b/src/bin/config/config.c @@ -278,52 +278,18 @@ _default_shortcuts_get() shortcut->holdable = Holdable; \ shortcuts = eina_list_append(shortcuts, shortcut); - /* No modifiers */ - ADD_SHORTCUT("Delete", 119, 0, "item.delete", false); - /* Ctrl- */ - ADD_SHORTCUT("slash", 61, CTRL, "separate_mode", false); - ADD_SHORTCUT("n", 57, CTRL, "new_theme", false); - ADD_SHORTCUT("o", 32, CTRL, "open_edj", false); - ADD_SHORTCUT("j", 44, CTRL, "import_edj", false); ADD_SHORTCUT("s", 39, CTRL, "save", false); - ADD_SHORTCUT("e", 26, CTRL, "export", false); ADD_SHORTCUT("w", 25, CTRL, "close", false); ADD_SHORTCUT("q", 24, CTRL, "quit", false); ADD_SHORTCUT("z", 52, CTRL, "undo", false); ADD_SHORTCUT("y", 29, CTRL, "redo", false); - ADD_SHORTCUT("1", 10, CTRL, "open_edj", false); - ADD_SHORTCUT("2", 11, CTRL, "tab.image_manager", false); - ADD_SHORTCUT("3", 12, CTRL, "tab.sound_manager", false); - ADD_SHORTCUT("4", 13, CTRL, "tab.style_manager", false); - ADD_SHORTCUT("5", 14, CTRL, "tab.colorclass_manager", false); - ADD_SHORTCUT("6", 15, CTRL, "animator", false); - ADD_SHORTCUT("equal", 21, CTRL, "zoom.in", false); ADD_SHORTCUT("minus", 20, CTRL, "zoom.out", false); - /* Alt- */ - ADD_SHORTCUT("w", 25, ALT, "part.add.swallow", false); - ADD_SHORTCUT("b", 56, ALT, "part.add.textblock", false); - ADD_SHORTCUT("t", 28, ALT, "part.add.text", false); - ADD_SHORTCUT("r", 27, ALT, "part.add.rectangle", false); - ADD_SHORTCUT("i", 31, ALT, "part.add.image", false); - ADD_SHORTCUT("p", 33, ALT, "part.add.proxy", false); - ADD_SHORTCUT("s", 39, ALT, "part.add.spacer", false); - ADD_SHORTCUT("g", 42, ALT, "part.add.group", false); - ADD_SHORTCUT("x", 53, ALT, "part.add.box", false); - - /* Ctrl-Shift- */ - ADD_SHORTCUT("o", 32, CTRL + SHIFT, "open_edc", false); - ADD_SHORTCUT("s", 39, CTRL + SHIFT, "save_as", false); - ADD_SHORTCUT("v", 55, CTRL + SHIFT, "property.visual_tab", false); - ADD_SHORTCUT("c", 54, CTRL + SHIFT, "property.code_tab", false); - ADD_SHORTCUT("l", 46, CTRL + SHIFT, "style.create", false); - /* Holdable keys. */ - ADD_SHORTCUT("Alt_L", 64, CTRL, "highlight.align.show", true); - ADD_SHORTCUT("Alt_L", 64, 0, "object_area.show", true); + ADD_SHORTCUT("Ctrl_L", 37, 0, "object_area.show", true); #undef ADD_SHORTCUT return shortcuts; diff --git a/src/bin/ui/shortcuts/shortcuts.c b/src/bin/ui/shortcuts/shortcuts.c index ef710f0..d069f5b 100644 --- a/src/bin/ui/shortcuts/shortcuts.c +++ b/src/bin/ui/shortcuts/shortcuts.c @@ -112,281 +112,10 @@ struct _Shortcut_Module Eina_List *keys; /* list of pointer to hash keys to be freed */ }; -/* -static void -_random_name_generate(char *part_name, unsigned int length) -{ - unsigned int i = 0; - - static const char CHARS[] = - "0123456789" - "ABCDEFGHIJKLMNOPQRSTUVWXYZ" - "abcdefghijklmnopqrstuvwxyz"; - - assert(part_name != NULL); - - for (i = 0; i < length - 1; ++i) - part_name[i] = CHARS[rand() % (sizeof(CHARS) - 1)]; - - part_name[length - 1] = 0; -} -*/ /*========================================================*/ /* SHORTCUTS CB FUNCTION */ /*========================================================*/ -#define PART_FUNCTIONALITY(TYPE, DATA) \ - SKIP_IN_ENVENTOR_MODE \ - Evas_Object *workspace = ap.workspace; \ - Evas_Object *widget_list = ui_block_widget_list_get(); \ - Style *style = NULL; \ - assert(style != NULL); \ - char name[9]; \ - _random_name_generate(name, 9); \ - if (workspace_edit_object_part_add(workspace, name, TYPE, DATA)) \ - ui_widget_list_part_add(widget_list, style, name); \ - history_diff_add(style->obj, PART_TARGET, ADD, name); \ - /* live_view_part_add(ap.live_view, ui_widget_list_selected_part_get(widget_list)); */ \ - project_changed(true); - -#define PART_ADD(TYPE, FUNC) \ -static Eina_Bool \ -_##FUNC##_part_add_cb(void) \ -{ \ - PART_FUNCTIONALITY(TYPE, NULL) \ - return true; \ -} - -/* Adding New Parts */ -/* -PART_ADD(EDJE_PART_TYPE_SWALLOW, swallow) -PART_ADD(EDJE_PART_TYPE_TEXTBLOCK, textblock) -PART_ADD(EDJE_PART_TYPE_SPACER, spacer) -PART_ADD(EDJE_PART_TYPE_TEXT, text) -PART_ADD(EDJE_PART_TYPE_RECTANGLE, rectangle) -PART_ADD(EDJE_PART_TYPE_PROXY, proxy) -PART_ADD(EDJE_PART_TYPE_GROUP, group) -PART_ADD(EDJE_PART_TYPE_BOX, box) -*/ -/* different adding is for image */ -/* -static void -_on_image_manager_done(void *data __UNUSED__, - Evas_Object *obj __UNUSED__, - void *event_info) -{ - char *selected = (char *)event_info; - - - if (!selected) return; - PART_FUNCTIONALITY(EDJE_PART_TYPE_IMAGE, selected); -} - -Eina_Bool -_image_part_choose_cb(void) -{ - Evas_Object *img_edit; - - - img_edit = image_manager_window_add(ap.project, SINGLE); - evas_object_smart_callback_add(img_edit, SIG_IMAGE_SELECTED, _on_image_manager_done, NULL); - return true; -} -*/ -TODO("move this code or some of it's part to Connector") -TODO("Change this shortcut function since there are not tabs inb widget_list anymore") -Eina_Bool -_item_delete_cb(void) -{ -/* SKIP_IN_ENVENTOR_MODE - Elm_Object_Item *glit = NULL; - Style *_style = NULL; - Evas_Object *nf = NULL; - Ewe_Tabs_Item *selected = NULL; - - COMMENT: if widget list is in focus. Delete part, style or layout. - nf = ui_block_widget_list_get(); - if ((nf) && (elm_object_focus_get(nf))) - { - selected = ewe_tabs_active_item_get(nf); - - COMMENT: try to delete layout - if ((evas_object_data_get(nf, "layouts_tab") == selected) && - ((!ap.project->current_style) || - (ap.project->current_style->__type != LAYOUT))) - { - evas_object_smart_callback_call(ap.block.left_top, "wl,layout,del", nf); - return true; - } - - COMMENT: try to delete style - else if ((evas_object_data_get(nf, "widgets_tab") == selected) && - ((!ap.project->current_style) || - (ap.project->current_style->__type != STYLE))) - { - nf = evas_object_data_get(nf, "nf_widgets"); - nf = elm_object_item_part_content_get(elm_naviframe_top_item_get(nf), - "elm.swallow.content"); - glit = elm_genlist_selected_item_get(nf); - if (glit) - { - _style = elm_object_item_data_get(glit); - if (_style->__type != WIDGET) - { - evas_object_smart_callback_call(ap.block.left_top, "wl,style,del", NULL); - return true; - } - } - } - } - - COMMENT: if state list is in focus - if (elm_object_focus_get(ap.block.state_list)) - { - evas_object_smart_callback_call(ap.block.state_list, "stl,state,del", NULL); - return true; - } - - COMMENT: try to delete part - evas_object_smart_callback_call(ui_block_widget_list_get(), "wl,part,del", NULL); -*/ - return true; -} - -static Eina_Bool -_separate_mode_change_cb(void) -{ - Evas_Object *workspace = tabs_current_workspace_get(); - - SKIP_IN_ENVENTOR_MODE - - if (!workspace) return false; - - double factor = workspace_zoom_factor_get(workspace); - if (fabs(factor - 1.0) > 0.001) - return false; - - /* - Eina_Bool sep = workspace_separate_mode_get(workspace); - workspace_separate_mode_set(workspace, !sep); - */ - - return true; -} - -TODO("Please remake this shortcut since there are no TABS anymore") -static Eina_Bool -_new_style_create_cb(void) -{ -/* - SKIP_IN_ENVENTOR_MODE - Elm_Object_Item *glit = NULL; - Style *_style = NULL; - Evas_Object *nf = ui_block_widget_list_get(); - Ewe_Tabs_Item *selected = ewe_tabs_active_item_get(nf); - - if (!elm_object_focus_get(nf)) return false; - if (!ap.project) return false; - - if ((evas_object_data_get(nf, "layouts_tab") == selected) && - ((!ap.project->current_style) || - (ap.project->current_style->__type != LAYOUT))) - { - evas_object_smart_callback_call(ap.block.left_top, "wl,layout,add", nf); - } - else if ((evas_object_data_get(nf, "widgets_tab") == selected) && - ((!ap.project->current_style) || - (ap.project->current_style->__type != STYLE))) - { - nf = evas_object_data_get(nf, "nf_widgets"); - nf = elm_object_item_part_content_get(elm_naviframe_top_item_get(nf), - "elm.swallow.content"); - glit = elm_genlist_selected_item_get(nf); - if (glit) - { - _style = elm_object_item_data_get(glit); - if (_style->__type != WIDGET) - evas_object_smart_callback_call(ap.block.left_top, "wl,style,add", NULL); - } - } -*/ - return true; -} - -TODO("Please remake this shortcut since there are no TABS anymore") -/* -static Eina_Bool -_style_delete_cb(void) -{ - SKIP_IN_ENVENTOR_MODE - Elm_Object_Item *glit = NULL; - Style *_style = NULL; - Evas_Object *nf = ui_block_widget_list_get(); - Ewe_Tabs_Item *selected = ewe_tabs_active_item_get(nf); - - if (!elm_object_focus_get(nf)) return false; - if (!ap.project) return false; - - if ((evas_object_data_get(nf, "layouts_tab") == selected) && - ((!ap.project->current_style) || - (ap.project->current_style->__type != LAYOUT))) - { - evas_object_smart_callback_call(ap.block.left_top, "wl,layout,del", nf); - } - else if ((evas_object_data_get(nf, "widgets_tab") == selected) && - ((!ap.project->current_style) || - (ap.project->current_style->__type != STYLE))) - { - nf = evas_object_data_get(nf, "nf_widgets"); - nf = elm_object_item_part_content_get(elm_naviframe_top_item_get(nf), - "elm.swallow.content"); - glit = elm_genlist_selected_item_get(nf); - _style = elm_object_item_data_get(glit); - if (_style->__type != WIDGET) - evas_object_smart_callback_call(ap.block.left_top, "wl,style,del", NULL); - } - return true; -} -*/ - -static Eina_Bool -_new_theme_cb(void) -{ - if (ap.colorsel) - evas_object_smart_callback_call(ap.colorsel, "dismissed", NULL); - tabs_menu_tab_open(TAB_HOME_NEW_PROJECT); - return true; -} - -/* -Eina_Bool -_open_edc_cb(void) -{ - if (!project_close()) - return false; - compile_dialog(); - return true; -} -*/ - -static Eina_Bool -_open_project_cb(void) -{ - if (ap.colorsel) - evas_object_smart_callback_call(ap.colorsel, "dismissed", NULL); - tabs_menu_tab_open(TAB_HOME_OPEN_PROJECT); - return true; -} - -static Eina_Bool -_import_edj_cb(void) -{ - if (ap.colorsel) - evas_object_smart_callback_call(ap.colorsel, "dismissed", NULL); - tabs_menu_tab_open(TAB_HOME_IMPORT_EDJ); - return true; -} - static Eina_Bool _save_cb(void) { @@ -395,53 +124,6 @@ _save_cb(void) } static Eina_Bool -_save_as_cb(void) -{ - if (!ap.project) return false; - project_export_develop(); - return true; -} - -static Eina_Bool -_export_cb(void) -{ - if (!ap.project) return false; - project_export_edc_project(); - return true; -} - -TODO("Remove? Or modify since we don't have EWE_TABS anymore?") -static Eina_Bool -_visual_tab_cb(void) -{ -/* - const Eina_List *tabs; - Evas_Object *nf; - - nf = ui_block_property_get(); - tabs = ewe_tabs_items_list_get(nf); - ewe_tabs_active_item_set(nf, eina_list_data_get(tabs)); -*/ - return true; -} - -TODO("Remove? Or modify since we don't have EWE_TABS anymore?") -static Eina_Bool -_code_tab_cb(void) -{ -/* - const Eina_List *tabs; - Evas_Object *nf; - - nf = ui_block_property_get(); - tabs = ewe_tabs_items_list_get(nf); - tabs = eina_list_next(tabs); - ewe_tabs_active_item_set(nf, eina_list_data_get(tabs)); -*/ - return true; -} - -static Eina_Bool _quit_cb(void) { ui_main_window_del(); @@ -457,75 +139,19 @@ _close_tab_cb(void) } static Eina_Bool -_style_manager_open_cb(void) -{ - if (ap.colorsel) - evas_object_smart_callback_call(ap.colorsel, "dismissed", NULL); - if (ap.project) - tabs_menu_tab_open(TAB_STYLE_EDITOR); - return true; -} - -static Eina_Bool -_image_manager_open_cb(void) -{ - if (ap.colorsel) - evas_object_smart_callback_call(ap.colorsel, "dismissed", NULL); - if (ap.project) - tabs_menu_tab_open(TAB_IMAGE_EDITOR); - return true; -} - -static Eina_Bool -_sound_manager_open_cb(void) -{ - if (ap.colorsel) - evas_object_smart_callback_call(ap.colorsel, "dismissed", NULL); - if (ap.project) - tabs_menu_tab_open(TAB_SOUND_EDITOR); - return true; -} - -static Eina_Bool -_colorclass_manager_open_cb(void) -{ - if (ap.colorsel) - evas_object_smart_callback_call(ap.colorsel, "dismissed", NULL); - if (ap.project) - tabs_menu_tab_open(TAB_COLORCLASS_EDITOR); - return true; -} - -static Eina_Bool -_animator_open_cb(void) -{ -/* if ((ap.project) && (tabs_current_group_get())) - animator_window_add(ap.project);*/ - return true; -} - -static Eina_Bool -_highlight_align_show_switch_cb(void) +_object_area_show_switch_cb(void) { Evas_Object *workspace = tabs_current_workspace_get(); + Eina_Bool flag; + SKIP_IN_ENVENTOR_MODE if (!workspace) return false; - SKIP_IN_ENVENTOR_MODE - Eina_Bool flag = workspace_highlight_align_visible_get(workspace); - workspace_highlight_align_visible_set(workspace, !flag); + flag = workspace_object_area_visible_get(workspace); workspace_object_area_visible_set(workspace, !flag); - return true; -} -static Eina_Bool -_object_area_show_switch_cb(void) -{ - Evas_Object *workspace = tabs_current_workspace_get(); - - SKIP_IN_ENVENTOR_MODE - if (!workspace) return false; - Eina_Bool flag = workspace_object_area_visible_get(workspace); + flag = workspace_highlight_align_visible_get(workspace); + workspace_highlight_align_visible_set(workspace, !flag); workspace_object_area_visible_set(workspace, !flag); return true; } @@ -625,40 +251,14 @@ typedef struct _Function_Set Function_Set; */ static Function_Set _sc_func_set_init[] = { - {"new_theme", _new_theme_cb}, -// {"open_edc", _open_edc_cb}, - {"open_edj", _open_project_cb}, - {"import_edj", _import_edj_cb}, {"save", _save_cb}, - {"save_as", _save_as_cb}, - {"export", _export_cb}, - {"property.visual_tab", _visual_tab_cb}, - {"property.code_tab", _code_tab_cb}, -/* {"part.add.swallow", _swallow_part_add_cb}, - {"part.add.textblock", _textblock_part_add_cb}, - {"part.add.text", _text_part_add_cb}, - {"part.add.rectangle", _rectangle_part_add_cb}, - {"part.add.image", _image_part_choose_cb}, - {"part.add.proxy", _proxy_part_add_cb}, - {"part.add.spacer", _spacer_part_add_cb}, - {"part.add.group", _group_part_add_cb}, - {"part.add.box", _box_part_add_cb},*/ -/* {"item.delete", _item_delete_cb}, this callback works unpredictable because of focus */ - {"separate_mode", _separate_mode_change_cb}, - {"style.create", _new_style_create_cb}, - {"highlight.align.show", _highlight_align_show_switch_cb}, - {"object_area.show", _object_area_show_switch_cb}, {"zoom.in", _zoom_in_cb}, {"zoom.out", _zoom_out_cb}, {"quit", _quit_cb}, {"close", _close_tab_cb}, {"undo", _undo_cb}, {"redo", _redo_cb}, - {"tab.style_manager", _style_manager_open_cb}, - {"tab.image_manager", _image_manager_open_cb}, - {"tab.sound_manager", _sound_manager_open_cb}, - {"tab.colorclass_manager", _colorclass_manager_open_cb}, - {"animator", _animator_open_cb}, + {"object_area.show", _object_area_show_switch_cb}, {NULL, NULL} }; static Eina_Hash *_sc_functions = NULL; --