Modify

Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#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 Ronald Parcinski, 11 years ago

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

comment:2 by yodolphe@…, 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:3 by Heiko Ellwanger, 10 years ago

Resolution: fixed
Status: newclosed

In 8463:

hellwanger: Autocomplete first Version (fix #508)

comment:4 by Heiko Ellwanger, 10 years ago

In 8468:

hellwanger: Autocomplete first Version (fix #508)

comment:5 by Torsten Riemer, 10 years ago

Resolution: fixed
Status: closedreopened

comment:6 by Ronald Parcinski, 10 years ago

Resolution: fixed
Status: reopenedclosed

In 8710:

try SumoSelect Autocomplete (fix #508)

comment:7 by Ronald Parcinski, 10 years ago

In 8711:

try SumoSelect Autocomplete (fix #508)

comment:8 by Ronald Parcinski, 10 years ago

In 8715:

updated sumoselect to modified version of 2.1.0 (fix #508 & #625)

comment:9 by Torsten Riemer, 10 years ago

Resolution: fixed
Status: closedreopened

comment:10 by Torsten Riemer, 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:11 by Ronald Parcinski, 10 years ago

zu 1.) r8725

zu 3.) r8725

comment:12 by Ronald Parcinski, 10 years ago

Resolution: fixed
Status: reopenedclosed

In 8725:

better sumoselect (fix #508)

comment:13 by Torsten Riemer, 10 years ago

Resolution: fixed
Status: closedreopened

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 #e9e9e9 (Zeile 25 sumoselect_mod.css):

.SumoSelect > .optWrapper > .options > li:hover {
  background-color: #e9e9e9;
  color:#af417e;
}

Sollten wir vielleicht noch vereinheitlichen?

Last edited 10 years ago by Torsten Riemer (previous) (diff)

comment:14 by Ronald Parcinski, 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:15 by Ronald Parcinski, 10 years ago

Resolution: fixed
Status: reopenedclosed

In 8728:

better sumoselect (fix #508)

comment:16 by Torsten Riemer, 10 years ago

Resolution: fixed
Status: closedreopened

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"

Version 1, edited 10 years ago by Torsten Riemer (previous) (next) (diff)

comment:17 by Ronald Parcinski, 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 Torsten Riemer, 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:19 by Ronald Parcinski, 10 years ago

Resolution: fixed
Status: reopenedclosed

In 8739:

better sumoselect (fix #508)

comment:20 by Torsten Riemer, 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:21 by Ronald Parcinski, 10 years ago

In 8746:

better sumoselect (fix #508)

comment:22 by anonymous, 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 Torsten Riemer, 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.

comment:24 by Ronald Parcinski, 10 years ago

In 8769:

better sumoselect (fix #508)

Modify Ticket

Action
as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.