search

msteudtn / Mat-O-Wahl

:de: Mat-O-Wahl - Ein einfach zu bedienender, freier Open Source Wahl-O-Mat Klon fuer jedermann ### :gb: :us: A simple to handle, free "Voting Advice Application" / "Electoral Compass" alternative
https://mat-o-wahl.de/
GNU General Public License v3.0
35 stars 15 forks source link

Issue 70 no details with textfilter #71

Closed msteudtn closed 2 years ago

msteudtn commented 2 years ago
msteudtn commented 2 years ago
fenglisch commented 2 years ago

Super! :)

Ich verstehe im Github GUI nicht ganz, wie ich diese Version testen kann. Gibt es dafür einen ZIP-Ordner oder muss ich die File Changes manuell in meine Instanz copy&pasten?

msteudtn commented 2 years ago

Das ist ne gute Frage.

Es gibt im Moment einen extra Entwicklungs-"tree" abseits des Master. https://github.com/msteudtn/Mat-O-Wahl/tree/issue-70-no-details-with-textfilter Dieser kann wieder normal als ZIP heruntergeladen werden.

Da es sich hier aber nur um drei relevante Dateien handelt (https://github.com/msteudtn/Mat-O-Wahl/pull/71/files), kannst du sie auch gern manuell aus dem Verzeichnis kopieren. :)

fenglisch commented 2 years ago

Habe es jetzt getestet:

Partei-Antworten werden nicht mehr angezeigt, wenn die Partei gefiltert wird

  • Funktioniert einwandfrei

ein einfaches Addon für Kontaktanfragen

  • Super, dass das mit den ausklappbaren Buttons und vor allem mit dem individuellen Ranking in der E-Mail geklappt hat!
  • Kleine Änderungsvorschläge (die wären vermutlich am besten in einem eigenen Ticket aufgehoben):
  • Dadurch, dass die beiden neuen Zeilen sich jeweils in einem eigenen div.mow-row-striped befinden, wird das Streifenmuster durcheinandergebracht. Besser wäre es, wenn die drei Zeilen (#resultsShortParty, #resultsShortPartyAddonContactsInResults, #nixTest) alle in einem div.mow-row-striped wären.
  • Super wäre es, wenn der Name des Vereins, bei dem ich als Nutzer auf den Button klicke, als Variable in die E-Mail (und ggf. auch schon in den Betreff) eingefügt werden könnte: "Ich habe Interesse an folgendem Verein: [NAME DES VEREINS]"

beim Ausklappen der Zusatzinformationen trotz Textfilter sollen die Antworten wieder zu sehen sein

  • Leider funktioniert es bei mir (https://mitwirk-o-mat.de/demo-v6-debug/) immer noch nicht
  • Sobald ich irgendeinen Filter gewählt habe (inkl. "Alle anzeigen"), lässt sich in der #resultsByPartyTable nichts mehr ausklappen
  • In der #resultsByThesisTable lässt sich zwar noch etwas ausklappen, aber es werden anfangs 0 Ergebnisse angezeigt, sondern nur die beiden Limit-Results-Buttons (erneut: egal, welchen Filter ich davor ausgewählt habe). Egal welchen der beiden Limit-Results-Buttons ich dann klicke, die Filterung wird überschrieben
  • Eine Kleinigkeit: Der erste Filter-Button hat gleich zu Anfang die Klasse .btn-primary, obwohl er nicht ausgewählt ist. Wenn der erste Filter "Alle anzeigen" ist, ergibt das Sinn, aber ich würde den "Alle anzeigen"-Button ganz nach hinten schieben, weil man den ja erst braucht, wenn man schon zuvor einen anderen Filter ausgewählt hat. Dann ergibt es keinen Sinn, wenn der erste Filter schon die Klasse .btn-primary hat
msteudtn commented 2 years ago

70 beim Ausklappen der Zusatzinformationen trotz Textfilter sollen die Antworten wieder zu sehen sein

Leider funktioniert es bei mir [...] immer noch nicht

Ich konnte es auf deinem Testsystem nachvollziehen aber nicht bei mir.

Kannst du mir bitte deine Test- und Systemdaten zukommen lassen, z.B. als ZIP-Archiv? Vielleicht ist meine Testumgebung zu "tolerant" oder es gab ein paar Anpassungen, die du übersprungen hast. Wir finden das raus. :)

fenglisch commented 2 years ago

Ich konnte es auf deinem Testsystem nachvollziehen aber nicht bei mir. Kannst du mir bitte deine Test- und Systemdaten zukommen lassen, z.B. als ZIP-Archiv? Vielleicht ist meine Testumgebung zu "tolerant" oder es gab ein paar Anpassungen, die du übersprungen hast. Wir finden das raus. :)

Jetzt klappt es komischerweise doch. Ich hatte schon den Verdacht, dass es entweder an meinen Anpassungen an der output.js oder der index.html liegen muss. Habe die erste durch das Original ersetzt und sofort ging es wieder. Auch nachdem ich alle meine Anpassungen manuell in die Original-output.js übertragen habe, ging es komischerweise noch. Naja, Hauptsache es funktioniert :-)

msteudtn commented 2 years ago

Ticket #70 wurde geschlossen. Die Änderungen werden nun in einem anderen Zweig weiter bearbeitet.