#508 closed Bug/Fehler (fixed)
In SumoSelect kann nicht mehr durch Eingabe eines Buchstaben an die entsprechende Stelle navigiert werden
| Reported by: | Torsten Riemer | Owned by: | somebody |
|---|---|---|---|
| Priority: | normal | Milestone: | modified-shop-2.0.0.0 |
| Component: | Admin | Version: | 2.0.0.0 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
Gibt man im Adminbereich bei ausgeklapptem SumoSelect einen Buchstaben auf der Tatstatur ein, so springt man nicht mehr automatisch an die entsprechende Stelle in der Dropdown-Liste.
Attachments (0)
Change History (24)
comment:1 by , 11 years ago
comment:2 by , 10 years ago
ja bitte raus mit denen. "beautifier" trifft es ganz gut.
Vor allem im Backend ist doch gute Performance Priorität, und dort muss man auch keinen potenziellen kunden mit Pseudo-iOS-Look "beeindrucken". lieber standardelemente mit erwartbarem verhalten.
Auch bei den Dropdowns und Checkboxen z.B. irritiert es mich total, dass der Cursor nicht mehr zu der "Pointer"-Hand wird. Das sind Standards, die sind in Ordnung und gelernt, die JS-"Verschönerungen" sind m.E. Augenwischerei. Lieber <label> -Elemente um alle inputs und Checkboxen drumrum mit ordentlich Padding, damit die Klickfläche groß wird und klickizackizacki ohne Gefummel.
comment:5 by , 10 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
comment:9 by , 10 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
comment:10 by , 10 years ago
Wie ich sehe sind durch das SumoSelect Update auf Version 2.1.0 in r8715 die Changesets r8463, r8468, r8710 & r8711 wieder revoked worden und die "head.php" damit wieder sauber. Prima!
Ich denke, dass das definitiv der bessere Ansatz ist!
Folgendes ist noch verbesserungswürdig:
1.) Anders als in der jquery.sumoselect - Demo wir bei Tastaturnavigation mit den Pfeiltasten die aktive Zeile nicht markiert.
2.) Auch in der Demo besteht das Problem, dass die Liste nicht nach unten scrollt, wenn man über die Pfeiltasten weiter nach unten geht, als Einträge in das Dropdown passen.
3.) Springt man über Eingabe eines Buchstabes im Dropdown an die entsprechende Stelle, dann wir die Prosition im Dropdown nicht aktualisiert.
comment:13 by , 10 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Prima! Dann fehlt nur noch die Behebung von Punkt 2.)
Ich denke wenn das gelöst ist, dann besteht auch das Problem nicht mehr, dass obwohl "Germany" ausgewählt ist das Dropdown bei "Afghanistan" anfängt, wenn man es öffnet.
P.S.: Die Farben im CSS für bei Navigation mit Tastatur ist #a1c0e4 (Zeile 26 sumoselect_mod.css):
.SumoSelect > .optWrapper > .options > li.sel {
background-color: #a1c0e4;
}
Bei Navigation über die Maus (hover) ist es aber #e0e0e0 (Zeile 22 sumoselect_mod.css):
.SumoSelect > .optWrapper > .options > li {
border-bottom: 1px solid #e0e0e0;
padding: 6px;
position: relative;
}
comment:14 by , 10 years ago
Beim Klick auf das Sumo springt der Cursor jetzt auf das vorselektierter Eintrag und der Eintrag ist auch markiert.
Die unterschiedlichen Farben sind auch im Original.
li.sel ist der derzeitige ausgewählte Eintrag
li.hover ist nur eine Postionsanzeige für die Maus, erst bei Klick wird die Auswahl ja übernommen
comment:16 by , 10 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Ich weiss, dass die Farben für die Tastatur-Eingabe aus dem Original CSS-Code des SumoSelects sind. Das heisst ja aber nicht, dass wir die Farben nicht an unser Corporate Design anpassen können.
Punkt 1.) & 2.) sind nun gefixt!
Durch r8728 scheint nun aber der Fix für Punkt 3.) aus r8725 nicht mehr zu funktionieren.
Nachtrag: Zusätzlich ist noch aufgefallen, dass die Tastatureingabe nur bis 3 Zeichen funktioniert. Gibt man "gree" ein, so springt er auf "East Timor" anstatt auf "Greece".
comment:17 by , 10 years ago
3.) funktioniert bei mir mit Win7 FF einwandfrei
Für die Tastatureingaben gibt es ein Timeout von 500ms. Da ist dann der 4. Buchstabe wieder der erste, oder schneller tippen.
comment:18 by , 10 years ago
Ich habe Punkt 3.) auch nochmal unter Win7 mit FF 40.0.3 getestet. Selbes Problem wie unter Mac.
Öffne mal das Dropdown und gib dann einen Buchstaben ein. im obersten Feld wird der Eintrag zwar ausgewählt, aber die Dropdown-Liste springt nicht an die entsprechende Stelle. Das funktionierte vorher aber (soweit ich mich erinnere) bereits durch deinen Fix aus r8725 und hörte erst mit r8728 wieder auf zu funktionieren.
Können wir die Timeout Zeit bitte auf 1000ms erhöhen?
comment:20 by , 10 years ago
Ich denke es ist nun perfekt! Die Farben der aktiven Zeile anzupassen macht evtl. doch nicht so viel Sinn, da die Browser die Zeile ohne SumoSelect auch blau markieren und man diese Farbe einfach gewohnt ist.
comment:22 by , 10 years ago
tastatur-sprung geht im aktuellen safari auf OSX immer noch nicht.
bitte weg mit dem sumoselect zeugs.
was für ein aufwand, damit die selects sich bitteschön genauso wie die VÖLLIG BEWÄHRTEN nativen eingabe-elemente des os- bzw. browser verhalten - aber halt in langsam. das alles nur damit es ein bisschen anders oder einheitlich ausschaut.
comment:23 by , 10 years ago
Das kann ich nicht nachvollziehen. Es funktioniert in Safari genau so wie in Firefox.
Das ganze lässt sich im Adminbereich unter "Konfiguration" -> "Adminbereich Optionen" -> "Adminbereich Styling" jederzeit deaktivieren.

Wegen der massiven Performance Probleme sollten diese Beautifier wieder ausgebaut oder erst einmal deaktiviert werden.
Sobald man eine Adminseite mit sehr vielen Checkboxen oder Selectfeldern hat gibt es JavaScript Laufzeitfehler