[EFL] back/forward keyboard shortcuts are non-intuitive
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Jun 2013 18:07:44 +0000 (18:07 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 28 Jun 2013 18:07:44 +0000 (18:07 +0000)
https://bugs.webkit.org/show_bug.cgi?id=118193

Patch by Balazs Kelemen <b.kelemen@samsung.com> on 2013-06-28
Reviewed by Christophe Dumez.

Changed shortcuts for go-back/go-forward to alt+left and alt+right.

* EWebLauncher/main.c:
(on_key_down):
* MiniBrowser/efl/main.c:
(on_key_down):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@152179 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/EWebLauncher/main.c
Tools/MiniBrowser/efl/main.c

index 2486daa..d03fd21 100644 (file)
@@ -1,3 +1,17 @@
+2013-06-28  Balazs Kelemen  <b.kelemen@samsung.com>
+
+        [EFL] back/forward keyboard shortcuts are non-intuitive
+        https://bugs.webkit.org/show_bug.cgi?id=118193
+
+        Reviewed by Christophe Dumez.
+
+        Changed shortcuts for go-back/go-forward to alt+left and alt+right.
+
+        * EWebLauncher/main.c:
+        (on_key_down):
+        * MiniBrowser/efl/main.c:
+        (on_key_down):
+
 2013-06-28  Roger Fong  <roger_fong@apple.com>
 
         Enable AVFoundation in Web Prefs for WinLauncher and DumpRenderTree on Windows.
index f41a4fd..5c833e9 100644 (file)
@@ -494,12 +494,14 @@ on_key_down(void *data, Evas *e, Evas_Object *obj, void *event_info)
         NULL
     };
     static int currentEncoding = -1;
-    Eina_Bool ctrlPressed = evas_key_modifier_is_set(evas_key_modifier_get(e), "Control");
+    const Evas_Modifier *mod = evas_key_modifier_get(e);
+    Eina_Bool ctrlPressed = evas_key_modifier_is_set(mod, "Control");
+    Eina_Bool altPressed = evas_key_modifier_is_set(mod, "Alt");
 
     if (!strcmp(ev->key, "Escape")) {
         closeWindow(app->ee);
-    } else if (!strcmp(ev->key, "F1")) {
-        info("Back (F1) was pressed");
+    } else if (!strcmp(ev->key, "Left") && altPressed) {
+        info("Back (Alt+Left) was pressed");
         if (ewk_view_back_possible(obj)) {
             Ewk_History *history = ewk_view_history_get(obj);
             Eina_List *list = ewk_history_back_list_get(history);
@@ -508,8 +510,8 @@ on_key_down(void *data, Evas *e, Evas_Object *obj, void *event_info)
             ewk_view_back(obj);
         } else
             info("Back ignored: No back history");
-    } else if (!strcmp(ev->key, "F2")) {
-        info("Forward (F2) was pressed");
+    } else if (!strcmp(ev->key, "Right") && altPressed) {
+        info("Forward (Alt+Right) was pressed");
         if (ewk_view_forward_possible(obj)) {
             Ewk_History *history = ewk_view_history_get(obj);
             Eina_List *list = ewk_history_forward_list_get(history);
index e9c0cea..1ca5155 100644 (file)
@@ -391,14 +391,16 @@ on_key_down(void *user_data, Evas *e, Evas_Object *ewk_view, void *event_info)
         NULL
     };
     static int currentEncoding = -1;
-    Eina_Bool ctrlPressed = evas_key_modifier_is_set(evas_key_modifier_get(e), "Control");
+    const Evas_Modifier *mod = evas_key_modifier_get(e);
+    Eina_Bool ctrlPressed = evas_key_modifier_is_set(mod, "Control");
+    Eina_Bool altPressed = evas_key_modifier_is_set(mod, "Alt");
 
-    if (!strcmp(ev->key, "F1")) {
-        info("Back (F1) was pressed");
+    if (!strcmp(ev->key, "Left") && altPressed) {
+        info("Back (Alt+Left) was pressed");
         if (!ewk_view_back(ewk_view))
             info("Back ignored: No back history");
-    } else if (!strcmp(ev->key, "F2")) {
-        info("Forward (F2) was pressed");
+    } else if (!strcmp(ev->key, "Right") && altPressed) {
+        info("Forward (Alt+Right) was pressed");
         if (!ewk_view_forward(ewk_view))
             info("Forward ignored: No forward history");
     } else if (!strcmp(ev->key, "F3")) {