-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Addon: Tooltips #81
Addon: Tooltips #81
Conversation
Auf Nutzer:innen-Feedback hin habe ich dieses kleine Addon erstellt, um zur Nutzung des "Doppelt gewichten"-Button anzuregen. Teilweise ist den Nutzer:innen nicht klar, was der bringt und dass sie den anklicken müssen, bevor sie ihre eigentliche Antwort auswählen.
Der Firefox in meiner Linux-Test-VM erzeugt keinen Tooltip. Der Chromium-Browser fügt aber alles ins DOM ein und zeigt es an. Ich vermute, dass das Problem um die Zeile 60 bei ... oder ich habe ein paar seltsame Einstellungen bei mir und bei dir funktioniert alles. :) Ich habe keine Tests auf mobilen Geräten gemacht. Abgesehen davon: Gute Idee und Umsetzung! 👍 :) |
Nachtrag: Solange #82 noch nicht umgesetzt ist, wäre ein Tooltip auf einen der beiden unteren Buttons am Ende auch noch eine nützliche Ergänzung. ... Das muss nicht gleich in der ersten Version drin sein, sondern reicht sicherlich auch später. :) |
Davor wurde Bubbling nur beim X-Button, also beim Schließen, verhindert, nicht beim Klicken auf einen anderen Teil des Tooltips. Dadurch könnte versehentlich der Doppelt Gewichten Button ausgelöst werden
Um Browser-Support zu gewährleisten (#81 (comment))
An das ganze Thema Browser-Support muss ich mich noch gewöhnen 🙈 |
Korrektur: Es lag nicht am Um das zu lösen, habe ich nach dem Eventlistener (load) noch einen Eventlistener (click) auf den "Los geht's"-Startbutton gelegt. Das ist keine optimale Lösung aber es erfüllt seinen Zweck.
|
- tooltipVotingDouble aus dem btnVotingDouble in den DIV-Container verschoben - tooltipResultsShort und tooltipResultsByThesis ergänzt
Ah verstehe. Ja, das klingt doch nach einem sinnvollen Workaround, habe ich gleich so übernommen. |
Ich hab jetzt zwei weitere Tooltips hinzugefügt. @msteudtn kannst du testen, ob das bei dir klappt? |
Der Tooltip bei "Doppelt gewichten" wird bei mir angezeigt 👍 - aber ohne eine Hintergrundfarbe. Das verwirrt ein wenig, wenn in der mobilen Ansicht die Abstimmen-Buttons durchschimmern. :( Die anderen Buttons werden nicht angezeigt. Laut Browser-Konsole kann die ID In den anderen Addons habe ich deshalb den
|
* Ändere default Wert von `TOOLTIP_BACKGROUND_COLOR` zu `var(--info)` (Bootstrap-Variable) * Korrigiere if expression in Zeile 224 * Erzeuge Variable `TOOLTIP_RESULTS_BY_THESIS_QUESTION_NUMBER`, die bestimmt, bei welcher Frage der Tooltip angezeigt werden soll
Ja, dafür muss oben die Variable
Der MutationObserver bezieht sich ja auf Der Der Also vermutlich ergibt das Addon erst nach einem Update der output.js Sinn |
- Kommentare am Ende der Klammern - "previousSibling()" wieder zurück auf "previousElementSibling()" zurück gesetzt. So funktioniert es in der `#resultsbyThesisTable`. - Noch kein Test mit Ergebnissen direkt in `#resultsShort` möglich
Auf Nutzer:innen-Feedback hin habe ich dieses kleine Addon erstellt, um zur Nutzung des "Doppelt gewichten"-Button anzuregen. Teilweise ist den Nutzer:innen nicht klar, was der bringt und dass sie den anklicken müssen, bevor sie ihre eigentliche Antwort auswählen.