Исправлена ошибка firefox'а с неработающими hotkeys(в русской раскладке)

Пожалуй, первый раз за все время на мнговение, сразу после прочтения заголовка, мне захотелось отправить чуток денег автору патча. Но быстро осознал что системы "ткни ссылку и заплати" нет (я всякими карточками ни разу не пользовался, один нал) ну и еще эта надись в конце новости - $500 и желание пропало :).

В любом случае это просто супер новость.

anonymous
(23.11.07 11:09:45 MSK)

Ссылка

Не прошло и семи лет.

birdie ★★★★★
(23.11.07 11:10:41 MSK)

Ссылка

Интересно, а в других национальных раскладках существует такая проблема с горячими кнопками? Если существует, тотоже решилась этим патчем? Или автор сделал костыль специально для русской раскладки?

Zubok ★★★★★
(23.11.07 11:15:16 MSK)

Ждём ебилдов

pento ★★★★★
(23.11.07 11:18:16 MSK)

Ссылка

Ответ на: комментарий от Zubok 23.11.07 11:15:16 MSK

>в других национальных раскладках существует такая проблема с горячими кнопками?

Да. Вообще во всех раскладках, отличающихся от Latin-1. НЯП, с двораком тоже проблема была.

>Или автор сделал костыль специально для русской раскладки?

Насколько я понял из обсуждения (я подписался на рассылку по этому багу), это не костыль, а именно решение. По какому принципу оно сделано - не помню.

Xellos ★★★★★
(23.11.07 11:19:23 MSK)

Вот так вот Linux превращается в обычную операционную систему, исчезают легенды firefox'а, KDE... Теперь о них будут вспоминать только старые линуксоиды за кружечкой пива... А вообще очень рад!

glebone ★
(23.11.07 11:21:34 MSK)

Ответ на: комментарий от Zubok 23.11.07 11:15:16 MSK

Вот меня то же итересует - это всё тот же патч, автор которого не раз говорил, что это жуткий костыль, и нужно переписывать всю подсистему работы с клавиатурой? Если да, то это только для русской "ЙЦУКЕН". И грош цена включению таких патчей.

anonymous
(23.11.07 11:21:58 MSK)

Простой советский парень Олег Крылов... Мы любим тебя.

anonymous
(23.11.07 11:27:51 MSK)

Ссылка

Ответ на: комментарий от anonymous 23.11.07 11:21:58 MSK

Блин, такой ведь баг был. Так все к нему привыкли, нам будет его не хватать... Теперь не о чем будет говорить в постах о FF... Может вернем всё в зад?

anonymous
(23.11.07 11:27:54 MSK)

Ссылка

Пойти нажраться на радостях!

svu ★★★★★
(23.11.07 11:27:55 MSK)

Ссылка

Ответ на: комментарий от Zubok 23.11.07 11:02:12 MSK

Конечно в фоксе. В epiphany, собранной на xulrunner'e такого бага никогда и не было.

scamelscrud ★
(23.11.07 11:31:45 MSK)

Ссылка

Ответ на: комментарий от glebone 23.11.07 11:21:34 MSK

Так можно плагин к firefox-у написать, который возвращает этот глюк на место! :)

KUser ★
(23.11.07 11:33:10 MSK)

для страждущих... патч на 2.0.0.9
не проверен, ибо собрать ещё не успел

--- mozilla/widget/src/gtk2/nsWindow.cpp.orig   2007-04-19 22:46:03.000000000 +0400
+++ mozilla/widget/src/gtk2/nsWindow.cpp        2007-11-23 11:35:57.000000000 +0300
@@ -1809,6 +1809,14 @@
     LOGFOCUS(("Done with container focus out [%p]\n", (void *)this));
 }
 
+inline PRBool
+is_latin_shortcut_key(guint aKeyval)
+{
+    return ((GDK_0 <= aKeyval && aKeyval <= GDK_9) ||
+            (GDK_A <= aKeyval && aKeyval <= GDK_Z) ||
+            (GDK_a <= aKeyval && aKeyval <= GDK_z));
+}
+
 gboolean
 nsWindow::OnKeyPressEvent(GtkWidget *aWidget, GdkEventKey *aEvent)
 {
@@ -1883,6 +1891,62 @@
         // clear isShift so the character can be inserted in the editor
 
         if (event.isControl || event.isAlt || event.isMeta) {
+            GdkEventKey tmpEvent = *aEvent;
+
+            // Fix for bug 69230:
+            // if modifier key is pressed and key pressed is not latin character,
+            // we should try other keyboard layouts to find out correct latin
+            // character corresponding to pressed key;
+            // that way shortcuts like Ctrl+C will work no matter what
+            // keyboard layout is selected
+            // We don't try to fix up punctuation accelerators here,
+            // because their location differs between latin layouts
+            if (!is_latin_shortcut_key(event.charCode)) {
+                // We have a non-latin char, try other keyboard groups
+                GdkKeymapKey *keys;
+                guint *keyvals;
+                gint n_entries;
+                PRUint32 latinCharCode;
+                gint level;
+
+                if (gdk_keymap_translate_keyboard_state(NULL,
+                                                        tmpEvent.hardware_keycode,
+                                                        (GdkModifierType)tmpEvent.state,
+                                                        tmpEvent.group,
+                                                        NULL, NULL, &level, NULL)
+                    && gdk_keymap_get_entries_for_keycode(NULL,
+                                                          tmpEvent.hardware_keycode,
+                                                          &keys, &keyvals,
+                                                          &n_entries)) {
+                    gint n;
+                    for (n=0; n<n_entries; n++) {
+                        if (keys[n].group == tmpEvent.group) {
+                            // Skip keys from the same group
+                            continue;
+                        }
+                        if (keys[n].level != level) {
+                            // Allow only same level keys
+                            continue;
+                        }
+                        if (is_latin_shortcut_key(keyvals[n])) {
+                            // Latin character found
+                            if (event.isShift)
+                                tmpEvent.keyval = gdk_keyval_to_upper(keyvals[n]);
+                            else
+                                tmpEvent.keyval = gdk_keyval_to_lower(keyvals[n]);
+                            tmpEvent.group = keys[n].group;
+                            latinCharCode = nsConvertCharCodeToUnicode(&tmpEvent);
+                            if (latinCharCode) {
+                                event.charCode = latinCharCode;
+                                break;
+                            }
+                        }
+                    }
+                    g_free(keys);
+                    g_free(keyvals);
+                }
+            }
+
            // make Ctrl+uppercase functional as same as Ctrl+lowercase
            // when Ctrl+uppercase(eg.Ctrl+C) is pressed,convert the charCode
            // from uppercase to lowercase(eg.Ctrl+c),so do Alt and Meta Key
@@ -1900,14 +1964,11 @@
            // bugs 50255 and 351310)
            if (!event.isControl && event.isShift &&
                (event.charCode < GDK_0 || event.charCode > GDK_9)) {
-               GdkKeymapKey k = { aEvent->hardware_keycode, aEvent->group, 0 };
-               guint savedKeyval = aEvent->keyval;
-               aEvent->keyval = gdk_keymap_lookup_key(gdk_keymap_get_default(), &k);
-               PRUint32 unshiftedCharCode = nsConvertCharCodeToUnicode(aEvent);
+               GdkKeymapKey k = { tmpEvent.hardware_keycode, tmpEvent.group, 0 };
+               tmpEvent.keyval = gdk_keymap_lookup_key(gdk_keymap_get_default(), &k);
+               PRUint32 unshiftedCharCode = nsConvertCharCodeToUnicode(&tmpEvent);
                if (unshiftedCharCode)
                    event.charCode = unshiftedCharCode;
-               else
-                   aEvent->keyval = savedKeyval;
            }
         }
     }

TOHbl4
(23.11.07 11:40:59 MSK)

Ссылка

Похожие темы