Projekt

Allgemein

Profil

« Zurück | Weiter » 

Revision a924abaf

Von Moritz Bunkus vor mehr als 7 Jahren hinzugefügt

  • ID a924abafa0da3e604d2b1b22e8041da82dae629a
  • Vorgänger 9b377bf4
  • Nachfolger a18b9450

PartPicker: Taskendrücke auf Shift, Ctrl & Alt ignorieren

Andernfalls wird schon beim Drücken eines der Modifier eine Suche
ausgelöst. Das kann besonders nervig sein, wenn man den Focus vom
Browser wegnimmt:

• Alt+Tab drücken, um zum anderen Fenster zu wechseln
• Schon bei Alt wird eine Suche ausgelöst.
• Durch den Focus-Verlust wird ein Blur-Event ausgelöst.
• Und schon ist der interne Zustand ungültig, und das Element wird rot
dargestellt.

Analoges passiert, wenn man z.B. mit Ctrl+F2 zu einem anderen virtuellen
Desktop wechselt.

Noch schlimmer ist, dass bei Shift+Tab ebenfalls zuerst eine Suche
ausgelöst und anschließend verhindert wird, dass der Focus das Element
verlässt.

Unterschiede anzeigen:

js/autocomplete_part.js
14 14
      RIGHT:  39,
15 15
      PAGE_UP: 33,
16 16
      PAGE_DOWN: 34,
17
      SHIFT:     16,
18
      CTRL:      17,
19
      ALT:       18,
17 20
    };
18 21
    var CLASSES = {
19 22
      PICKED:       'partpicker-picked',
......
190 193
      select: function(event, ui) {
191 194
        set_item(ui.item);
192 195
      },
196
      search: function(event, ui) {
197
        if ((event.which == KEY.SHIFT) || (event.which == KEY.CTRL) || (event.which == KEY.ALT))
198
          event.preventDefault();
199
      }
193 200
    });
194 201
    /*  In case users are impatient and want to skip ahead:
195 202
     *  Capture <enter> key events and check if it's a unique hit.
......
223 230
          });
224 231
          return false;
225 232
        }
226
      } else {
233
      } else if ((event.which != KEY.SHIFT) && (event.which != KEY.CTRL) && (event.which != KEY.ALT)) {
227 234
        state = STATES.UNDEFINED;
228 235
      }
229 236
    });

Auch abrufbar als: Unified diff