Skip to content
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

Mailto-Link in der Ergebnisanzeige #65

Closed
fenglisch opened this issue Jul 19, 2022 · 14 comments
Closed

Mailto-Link in der Ergebnisanzeige #65

fenglisch opened this issue Jul 19, 2022 · 14 comments
Assignees

Comments

@fenglisch
Copy link
Collaborator

fenglisch commented Jul 19, 2022

Die Idee habe ich aus dem Klimafinder (https://klimafinder.netlify.app/). Wenn du alle Fragen beantwortest (ganz egal wie), kommst du zum Ranking, klickst dann auf irgendein Angebot und siehst den Button "Connect me". Dahinter steckt ein komplexer mailto-Link. Wenn du draufklickst, öffnet sich dein Standard-Mail-Programm mit einer vorformulierten Mail mit folgendem Inhalt:

  • Variable Empfänger:in(nen)
  • Variabler Betreff
  • Standard-Anschreiben mit einzelnen variablen Inhalten
  • Liste, mit welchen Angeboten ich welche Übereinstimmung hatte (also mein Ranking in Textform)

Für einen Wahlhelfer, also den klassischen Wahl-O-Mat Use Case, braucht es so etwas natürlich nicht, aber für den Mitwirk-O-Mat wäre das umso sinnvoller. Die Nutzer:innen sollen ja, nachdem sie das Ranking sehen, nicht direkt abspringen, sondern Kontakt aufnehmen. So eine vorformulierte E-Mail würde die Hürde dafür sicherlich verringern.

Es könnte auch eingestellt werden, an wen die E-Mail gehen soll, und davon abhängig natürlich der Inhalt:

  • Direkt an den Verein, um zu fragen, wie/wo/wann man sich beteiligen kann (dann bräuchte es die Ranking-Liste mit den Übereinstimmungen mit den anderen Vereinen nicht in der Mail)
  • An die Freiwilligenagentur, um Interesse an einem konkreten Angebot mitzuteilen und einen Beratungstermin hierzu zu vereinbaren. Gleichzeitig wird das Ranking (z. B. die Top 10) mitgeschickt, damit die FWA mehr über die Person weiß und gezielter beraten kann, nicht nur zu dem einen konkreten Angebot
    • Hintergrund: Für viele FWA's ist die klassische Beratung trotz digitaler Tools ein zentraler Baustein und einige FWA's müssen auch nachweisen, dass sie jährlich soundsoviel Beratungsgespräche geführt haben, um ihre Förderung zu bekommen. Der Mitwirk-O-Mat sollte diesen FWA also mehr und nicht weniger Termine bescheren.

Ich könnte mir das als weiteres Add-On vorstellen, mit einer Datei, wo ich die ganzen Strings inkl. Variablen eingebe (Betreff und Anschreiben). Hier wähle ich auch aus, ob die Mail an eine (in der parteien.csv zu hinterlegende) E-Mail-Adresse des Vereins oder an die (in der Add-On-Datei zu hinterlegende) E-Mail-Adresse der Freiwilligenagentur gesendet werden soll. Daraus werden dann die mailto-Links für die Buttons generiert, die in der Ergebnistabelle angezeigt werden.

Was ich mir schwer vorstelle, ist die Übertragung des Rankings in den E-Mail-Text. Wenn das nicht gehen sollte, wäre das zwar schade, aber das Feature wäre trotzdem sinnvoll (theoretisch könnte man bitten, dass der Nutzer ein Screenshot vom Ranking macht und in die Mail einfügt).

Ich weiß nicht, ob der Klimafinder Open Source ist, vielleicht würde ein Blick in deren Code auch helfen.

@msteudtn
Copy link
Owner

Hallo,

eine coole Idee.

Technisch wäre vielleicht wirklich ein Add-On die sauberste Lösung. Ansonsten müsste man anfangen die parteien.csv resp. vereine.csv mit unendlich vielen Spalten aufzubohren. :(

Das Add-On hätte in der einfachsten Ausführung einige Arrays der Art:

emailadresse = ["kontakt@verein.de", "fwa@ort.de", ...]
telefonnummer = ["", "0123 - 456789", ...]
betreff =  ["Anfrage XY", "Anfrage ABC", ...]
inhalt = ["Hallo, ...", ...]

... oder in der schönen Variante hätte man eine zusätzliche CSV-Textdatei mit den Inhalten:

emailadresse telefonnummer betreff inhalt
kontakt@verein.de . Anfrage XY Hallo, ...
fwa@ort.de 0123 - 456789 Anfrage ABC ...

Das SOLLTE auch gar nicht so umständlich sein, da /SYSTEM/GENERAL.JS -> fnReadCsv(csvFile,fnCallback) von Haus aus eine Nachfolge-Funktion (nach dem Einlesen einer CSV-Datei) erwartet. Diese Folge-Funktion wäre dann das Schreiben der Daten in die Ergebnisse. :)

Das Thema "Versenden der Ergebnisse" würde ich erst mal nach hinten verschieben. Das kommt sicherlich auch darauf an, ob eher die Freiwilligenagentur (FWA) oder doch der Verein angesprochen werden möchte.

Übrigens könnte man mit der Telefonnummer auch Messenger-Dienste über einen weiteren Button einbinden. Ein Beispiel wäre z.B. Whatsapps "Click to Chat" https://faq.whatsapp.com/452366545421244/?locale=de_DE

@msteudtn msteudtn self-assigned this Jul 19, 2022
@fenglisch
Copy link
Collaborator Author

Danke für die schnelle Antwort, dass sind ja gute Neuigkeiten!

Mit Hinblick auf die Skalierbarkeit stelle ich es mir relativ aufwändig und fehleranfällig vor, wenn die Plugin-Datei so viele Daten enthalten muss (egal ob als Array oder als CSV). Bei 100 Vereinen wäre das ja sehr müßig.

Die Telefonnummer bräuchten wir dafür nicht, das macht das Ganze schon mal einfacher. Für die E-Mail-Adressen wäre es aber am besten, wenn die (irgendwie) mit in der parteien.csv stehen würden. Damit ließen sich die Daten in einem Rutsch eintragen und auch deutlich einfacher pflegen, z. B. wenn man einen Verein wieder entfernt, wäre die E-Mail-Adresse direkt weg und alle anderen E-Mail-Adressen wären weiterhin richtig zugeordnet.

Als zweitbeste Option kann ich mir vorstellen, dass die E-Mail-Adressen in einer zweiten CSV eingetragen werden, die dann nichts anderes als diese E-Mail-Adressen enthält. Das hätte dann den Nachteil, dass sich anhand der Zeilennummer nicht so leicht überprüfen lässt, welche E-Mail-Adresse zu welchem Verein gehört, aber das ließe sich ja berechnen: Zeilennummer der Vereinsabkürzung in der parteien.csv / (Anzahl der Fragen + 7) + 1 = Zeilennummer der E-Mail-Adresse in der anderen CSV (oder so ähnlich)

Der Betreff und das Anschreiben sollten aber meiner Meinung nach weder in einem Array noch in einer CSV stehen, sondern in der Plug-In-Datei einfach als String hinterlegt. Betreff und Inhalt der Mail sollen ja nicht jedes Mal abweichen und das wäre auch viel zu aufwändig, das zu pflegen. Idealerweise würde der String Variablen enthalten (Name des Vereins, ggf. sogar Prozent der Übereinstimmung, bräuchte es aber nicht unbedingt, wenn der Verein direkt angeschrieben wird)

Die Idee mit Click-to-Chat finde ich sehr charmant und denke, dass das die Nutzer:innen abholen würde, aber gleichzeitig bezweifle ich, dass die Freiwilligenagenturen oder Vereine selbst das managen könnten, also dass sie dienstlich WhatsApp nutzen (ist ja auch mit Hinblick auf die DS-GVO ein no go).

@msteudtn
Copy link
Owner

Hallo,

Skalierbarkeit

ja, ab einer gewissen Anzahl wird das unübersichtlich.

Die Telefonnummer bräuchten wir dafür nicht

Ich würde die Telefonnummer noch nicht ad acta legen. Es gibt bestimmt viele Leute, die keinen Mailzugang mehr auf ihrem Smartphone eingerichtet haben. Da bleibt nur noch der Anruf oder der Messenger-Dienst.
Außerdem möchte ich das Add-On gern langfristig anlegen, falls andere Anwender doch die Telefonnummer brauchen. :)

Für die E-Mail-Adressen wäre es aber am besten, wenn die (irgendwie) mit in der parteien.csv stehen würden

Dann stehe ich - ähnlich wie bei den Tickets #44 und #53 (fragen.csv) - vor der Frage ob man weitere Spalten oder Zeilen hinzufügen muss. :(

Alternativ würden die Informationen in einem vorhandenen Textfeld (wie der Partei-Beschreibung) versteckt und müssten dann "herausgefischt" werden. z.B. innerhalb eines <!-- HTML-Kommentars --> oder etwas ähnlichem.

Beispiel der normalen parteien.csv: https://github.com/msteudtn/Mat-O-Wahl/blob/abbdfccbc29c2f7b44b898d162c651c58532dc7c/data/Obsthausen_Parteien.csv

Als zweitbeste Option kann ich mir vorstellen, dass die E-Mail-Adressen in einer zweiten CSV eingetragen werden

Das wäre im Moment mein Favorit. Die Reihenfolge müsste dann wie in der parteien.csv sein. Aber der Aufbau wäre so, wie im Kommentar weiter oben. #65 (comment)
Also müsste man keinen Abstand lassen, sondern kann die Kontaktdaten "leicht" wiederfinden.
Es könnte zusätzlich noch eine Spalte "Beschreibung" geben. Wenn sie ausgefüllt ist, weiß man auch, welcher Verein hinter Karl-Theodor.von-und-zu-1971@t-online.de steckt.

Der Betreff und das Anschreiben sollten aber meiner Meinung nach weder in einem Array noch in einer CSV stehen

Mein erstes Verständnis war, dass je nach Bedarf die Vereine oder die Freiwilligenagenturen (FWA) angeschrieben werden. Aber ein allgemeines Anschreiben reduziert natürlich den Aufwand. :)

Die Idee mit Click-to-Chat ...

siehe oben mit der Telefonnummer. :)
Aber ja, den genauen Umgang mit Messenger-Diensten und der DSGVO in den FWA kenne ich nicht.

@fenglisch
Copy link
Collaborator Author

fenglisch commented Jul 21, 2022

Telefonnummer: Du hast recht, vermutlich ist es sinnvoll, das Add-On so zu bauen, dass die Telefonnummern direkt mit hinterlegt werden können. Die kann dann ja auch für einen Anruf-Button genutzt werden, da viele Freiwilligenagenturen und einige Vereine ja Sprechzeiten haben (und wenn man außerhalb der Sprechzeiten anruft, kann man ja auf die AB quatschen). Und vielleicht gibt es ja auch Freiwilligenagenturen, die man über einen Messenger anschreiben kann, dann wäre Click-to-chat natürlich super.

E-Mail-Adressen in parteien.csv: Das in ein anderes Feld reinzuwurschteln, fände ich nicht gut. Das machen wir jetzt ja schon mit den Filterkriterien. Da ist für uns der Aufwand auch ziemlich groß, den LimeSurvey-Export so umzuformatieren, dass am Ende die Filter-Kriterien mit im Beschreibungs-String enthalten sind, aber dafür haben wir jetzt eine Lösung gefunden. Mit jeder weiteren Information wird das aber unübersichtlicher und komplizierter.

Am besten wäre es wohl, wenn die parteien.csv nicht aus zwei Spalten und n Zeilen bestehen würde (n = Zahl der Parteien * (Anzahl der Fragen + 6)), sondern aus x Spalten (x = 5 + Anzahl der Fragen x 2) und y Zeilen (y = Anzahl der Parteien). Anders ausgedrückt: Für jede Partei hat eine Zeile und die Daten zur Parteien sind dann in mehreren Spalten. Dann könnte man ja nach meiner Vorstellung hinter den "Pflicht-Spalten" (Allgemeine Angaben und Antworten auf die Fragen) noch weitere optionale Spalten einfügen, die nur ausgelesen werden, wenn ein Add-On aktiviert ist, z. B. die Filter-Kriterien oder E-Mail-Adressen.

E-Mail-Adressen in separater CSV: Ja genau, ein Abstand wäre nicht sinnvoll. Das über die Beschreibung abzugleichen, wäre bei vielen Vereinen aber auch nicht unbedingt das einfachste. Ich hab jetzt aber die Formel zur Umrechnung der Zeilennummer des Vereins zur Zeilennummer der E-Mail-Adresse:
Zeile der E-Mail-Adresse = (Zeile des Vereins - 1) / (Anzahl der Fragen + 6) + 1
Umgestellt: Zeile des Vereins = (Zeile der E-Mail-Adresse - 1) x (Anzahl der Fragen +6) +1

Damit lässt sich schnell finden, welche E-Mail-Adresse aktualisert bzw. gelöscht werden muss, wenn sich bei einem der Vereine etwas ändert. Es ist dann immer noch fehleranfälliger, als wenn das alles direkt in der parteien.csv wäre, aber da müssen wir einfach gut aufpassen. Jetzt, wo ich drüber nachdenke, wäre es vermutlich doch nicht schlecht, sicherheitshalber den Namen des Vereins auch in der E-Mail-Adressen-CSV einzutragen, auch wenn er da gar nicht ausgelesen wird.

Die Struktur wäre dann wie folgt:

emailadresse telefonnummer Name
kontakt@verein.de . ABC
verein-b@web.de 0123 - 456789 XYZ

FWA oder Verein anschreiben: Nach meiner Vorstellung reicht es, wenn das zentral in der Add-On-Datei vorgegeben ist, an wen die E-Mail gehen soll: Immer an den jeweiligen Verein oder immer an die FWA. Wenn die Mails an die FWA gehen, braucht es die E-Mail-Adressen-CSV gar nicht, sondern die E-Mail-Adresse der FWA wird in der Add-On-Datei hinterlegt. So oder so ist das Anschreiben jeweils gleich: Es werden entweder alle Vereine mit demselben Text angeschrieben (ggf. mit Variablen) oder es wird die FWA mit dem immer gleichen Text angeschrieben (Variablen wären hier wichtig). Das sind dann natürlich zwei verschiedene Texte.

msteudtn added a commit that referenced this issue Jul 23, 2022
@msteudtn
Copy link
Owner

Ich habe ein paar Tests gemacht und bin darüber gestolpert, dass die gesamte Ergebniszeile mit einer Klickfunktion belegt ist. Diese öffnet die Beschreibung der jeweiligen Partei / des Vereins. Hintergrund war, dass man auf dem Smartphone mit seinen Wurstfingern :) einfacher die Zeile antippt als den Text.

Mat-o-Wahl-Kontaktbuttons

Das bedeutet, dass (im Moment) beim Klick auf die Buttons "E-Mail" oder "Telefon" jedes mal die Beschreibung geöffnet / geschlossen wird.

Man könnte die Klickfunktion wieder reduzieren oder eine Zwischenzeile einfügen. Eventuell könnte die Zwischenzeile sich zusammen mit der Beschreibung öffnen.

Die Betaversion des Add-Ons liegt hier: https://github.com/msteudtn/Mat-O-Wahl/blob/issue-65-addon-contact-buttons/extras/addon_contacts_in_results.js Das Add-On kann noch nicht viel. Es gibt nur eine E-Mailadresse, eine Telefonnummer und keinen E-Mailtext.

@fenglisch
Copy link
Collaborator Author

Wie cool!

Ich glaube, es wäre sinnvoller, wenn die Buttons auch erst versteckt wären und mit ausgeklappt werden. Wenn ich über einen Verein nicht mehr erfahren möchte, dann möchte ich ja wegen dieses Vereins auch keine E-Mail versenden oder einen Anruf tätigen.

Das Problem mit dem Ein- und Ausklappen bliebe dann vermutlich trotzdem, also dass die Beschreibung wieder einklappt, wenn ich auf den Button klicke. Aber dafür finden wir dann bestimmt eine Lösung.

Das Add-On ist natürlich noch nicht fertig, aber schon mal eine Anregung zu den Telefon-Buttons: Es wäre cool, wenn in der Add-On-Datei eingestellt werden kann, welcher Button angezeigt werden soll, also z. B.:

var eMail = 1;
var telephone = 0;

Damit könnte die Telefon-Option ganz aktiviert werden, falls die Freiwilligenagentur das nicht will. Und selbst wenn die Telefon-Option aktiviert ist, könnte man den Telefon-Button dann über einen CSS @media-query auf Desktops verstecken, weil vermutlich die wenigsten ein Telefon-Programm auf dem PC haben, sondern dann eher eine E-Mail schreiben. Oder die Telefonnummer wird angezeigt und kann mit dem Handy abgetippt werden. Gleichzeitig könnte die ausgeschriebene Telefonnummer mit einem Tel-Link hinterlegt sein, sodass beides ermöglicht wird, abtippen oder draufklicken. Aber da würde ich jetzt nicht zu viel Hirnschmalz und Arbeit reinstecken, weil ich schätze, dass diese Optionen sowohl von den Nutzer:innen (viele sind ja telefonier-scheu) als auch von den Vereinen und Freiwilligenagenturen weniger nachgefragt sein wird als die E-Mail-Option.

msteudtn added a commit that referenced this issue Jul 23, 2022
* Verbesserungen für Addon im Ticket #65 
* Option zum (De)-Aktivieren der Buttons
* Variable für den Betreff und Text der E-Mail
* neue Zwischenzeile in der Ergebnisliste inkl. Klickfunktion auf der vorhandenen Ergebniszeile
msteudtn added a commit that referenced this issue Jul 23, 2022
* Weitere Verbesserungen für Ticket #65 
* Button erschien nur in der ersten Zeile wegen einer falsch zugeordneten [i] / [j]-Variable in einer FOR-Schleifen
* Die neue Zwischenzeile mit den Buttons hat nun auch einen hübschen Rahmen
* In der E-Mail werden die Ergebnisse mit angehängt.
msteudtn added a commit that referenced this issue Aug 1, 2022
* Testzeile ohne Funktion für andere Addons
* Parteiname wird in den E-Mailbetreff übernommen
* #65
@msteudtn
Copy link
Owner

msteudtn commented Aug 1, 2022

Zitat aus #71 (comment)

Kleine Änderungsvorschläge [...]

  • 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.

Für das Ein- und Ausblenden "zähle" ich die CSS-Klasse row in den <DIV>s (in der ich dann auch mow-row-striped unterbringe). Wenn ich eine zusätzliche Zeile einfüge (oder auch wegnehme), kommt am Ende alles wieder durcheinander. :)

Abgesehen davon, besteht das "Streifenproblem" ja auch beim Textfilter extras/addon_results_textfilter_by_button.js. Dort werden die Zeilen ebenfalls so eingeblendet, wie sie ursprünglich vorgesehen waren.

Lösungsansätze:

Man könnte den Rahmen in den extra Zeilen auch einfach entfernen, indem man die CSS-Klasse weglässt. Ich persönliche finde aber, dass die Inhalte dann etwas verloren in der Luft hängen.

Alternativ müsste nach jedem Seiten-Rendern nochmal über die Tabelle gegangen werden und geprüft werden, ob die vorherige Zeile hell oder dunkel war. Das wäre aber etwas für ein sehr langes, verregnetes Wochenende. :)

  • 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]"

Im Entwicklungszweig issue-65-addon-contact-buttons/gibt es nun eine aktualisierte Addon-Datei.
https://github.com/msteudtn/Mat-O-Wahl/blob/issue-65-addon-contact-buttons/extras/addon_contacts_in_results.js

Achtung: Es ist das Addon mit der zusätzlichen "Testzeile ohne Funktion".

@fenglisch
Copy link
Collaborator Author

Ich hab deine Antwort nicht wirklich verstanden... Also das mit dem Streifenproblem beim Filtern schon, das ist mir auch aufgefallen, aber finde ich nicht so schlimm. Dafür die Seite neu rendern wäre echt übertrieben.

Ich stelle mir das mit den neuen Zeilen so vor:
grafik

Hab das im Browser so zusammengebastelt, das Ein- und Ausklappen hat funktioniert:

<div class="row border rounded mow-row-striped" id="resultsShortParty0" role="row">
   <div class="col col-10 col-md-7" role="cell">
      <strong>Kulturstadt e.&nbsp;V.</strong> (⇗ <a href="https://beispiel.de" target="_blank" alt="Link: Kulturstadt e.&nbsp;V." title="Link: Kulturstadt e.&nbsp;V.">KS</a>)
      <p style="cursor: pointer;"> ↓;  <span id="resultsShortPartyDescriptionDots0" style="display: none;">...</span><span id="resultsShortPartyDescription0" style="">&nbsp;Wir bringen Musik, Theater und Tanz nach Neustadt.<br><small>#12345</small></span> </p>
   </div>
   <div class="col col-2 col-md-1" role="cell"><img src="data/logos/ks.png" class="rounded img-fluid" alt="Logo Kulturstadt e.&nbsp;V."></div>
   <div class="col col-12 col-md-4" role="cell">
      <div class="progress">
         <div class="progress-bar bg-success" role="progressbar" id="partyBar0" style="width:100%;" aria-valuenow="100" aria-valuemin="0" aria-valuemax="100">100% (1 / 1)</div>
      </div>
   </div>
   <div id="resultsShortPartyAddonContactsInResults0" style="flex: 1 1 100%;" class="row border rounded">
      <div class="col" style="text-align: center;">  <a href="mailto:..." role="button" class="btn btn-sm btn-success">Kontakt per E-Mail</a> </div>
      <div class="col" style="text-align: center;">  <a href="tel:+49123456789" role="button" class="btn btn-sm btn-success" aria-pressed="true">Kontakt per Telefon</a> </div>
   </div>
   <div id="nixTest0" style="flex: 1 1 100%;" class="row border rounded">
      <div class="col"> Testzeile ohne Funktion j: 0 </div>
   </div>
</div>

Also das ist garantiert nicht sauber, aber ginge das so in die Richtung?

@fenglisch
Copy link
Collaborator Author

Vielleicht könnte man auch den ganzen oberen Teil in ein eigenes DIV packen und die Klick-Funktion zum Ein- und Ausklappen nur auf dieses DIV legen, damit es beim E-Mail-Button nicht passiert

Also die drei div.col mit der Beschreibung, dem Logo und dem Balken nochmal in ein eigenes div.row und dieses div.row dann zusammen mit dem neuen div#resultsShortPartyAddonContactsInResults in das bestehende div.mow-row-striped

@msteudtn
Copy link
Owner

msteudtn commented Aug 2, 2022

#65 (comment) Hab das im Browser so zusammengebastelt, das Ein- und Ausklappen hat funktioniert [...]

Wenn ich das richtig interpretiere, liegen alle Informationen (Beschreibungen, Bilder, Prozentbalken, Kontaktdaten, Testzeile) unterhalb der jeweiligen DIV resultsShortParty0, resultsShortParty1 usw. Das war auch mein erster Gedanke (Siehe #65 (comment)) aber auf dem entsprechenden DIV liegt bereits eine Klickfunktion. Dadurch konnte man die Buttons nicht erreichen. :(

Vielleicht könnte man auch den ganzen oberen Teil in ein eigenes DIV packen und die Klick-Funktion zum Ein- und Ausklappen nur auf dieses DIV legen, damit es beim E-Mail-Button nicht passiert

Das wäre langfristig vielleicht eine Lösung. Im Moment würde es aber noch mehr Komplexität in das Addon bringen (Prüfen, ob es die "Hauptzeile" ist, abzählen wie viele Extrazeilen es gibt, ausblenden der entsprechenden Zeilen) - besonders, da Ticket #69 noch offen ist. :)

Lange Rede, kurzer Sinn: Das Thema "Streifen beim Filtern" würde ich gern auf "später" verschieben.

@fenglisch
Copy link
Collaborator Author

fenglisch commented Aug 3, 2022

Genau, als aktuell sind die drei div.col mit der Beschreibung, dem Logo und dem Balken jeweils direkte children der div#resultsShortPartyX. Die div#resultsShortPartyAddonContactsInResults und die div#nixTestX sind aktuell keine children der div#resultsShortPartyX, sondern folgen direkt dahinter.

Mein Vorschlag wäre nun, die drei div.col in eine zusätzliche div.row zu packen, z. B. mit der ID #resultsShortPartyGeneralX, unabhängig davon, ob das Addon aktiviert ist oder nicht. Die Klickfunktion zum Ausklappen könnte dann nur auf dieses #resultsShortPartyGeneralX gelegt werden und nicht auf die ganze div#resultsShortPartyX. Wenn das E-Mail-Addon nicht aktiviert ist, macht das keinen Unterschied und das zusätzliche div wäre nicht notwendig, aber stört auch nicht, aber sobald das Addon aktiviert ist, wären alle drei divs (#resultsShortPartyGeneralX, div#resultsShortPartyAddonContactsInResults und div#nixTestX) direkte children der div#resultsShortPartyX, was den gewünschten Effekt sowohl bei den Streifen als auch bei der Klickfunktion ergeben würde (wenn ich mir das richtig vorstelle)

@fenglisch
Copy link
Collaborator Author

Nice, das sieht super aus! Auch mit dem Namen des Vereins im E-Mail-Betreff.

Ein Bug ist mir noch aufgefallen, sicherlich ausgelöst durch den Konflikt zwischen Limit-Results- und Filter-Addon. Vermutlich würde er kaum auffallen, weil er nur auftritt, wenn man auf spezielle Weise mit der Ergebnistabelle interagiert. Vielleicht gibt es aber auch eine weniger umständlichere Abfolge als diese:

  1. Öffne https://mitwirk-o-mat.de/issue-65 (die Original-Version mit folgenden Addons aktiviert: "extras/addon_results_textfilter_by_button.js", "extras/addon_check_iframe_resize_client.js", "extras/addon_limit_results.js", "extras/addon_contacts_in_results.js")
  2. Beantworte alle Fragen mit "Ja", um zum Ergebnis zu kommen.
  3. Klicke auf alle Ergebnisse, um die Beschreibung und die Kontakt-Buttons auszuklappen.
  4. Klicke auf "Südfrüchte anzeigen"
  5. Klicke auf "Mehr Ergebnisse anzeigen"

-> Bei der Apfelpartei wird die vollständige Beschreibung angezeigt, aber nicht die Buttons. Wenn ich auf die Beschreibung klicke, wird diese eingeklappt, aber die Buttons erscheinen (bei den Neutralen ist das auch so, aber da fällt es kaum auf). Je nachdem, wie die Fragen beantwortet und welcher Filter gewählt wird, passiert es auch bei den anderen Parteien.

Da das sicherlich nur sehr selten auftritt und auch dann noch zu benutzen ist, hat die Behebung dieses Bugs meiner Meinung nach nicht die höchste Priorität. Ich wollte dich aber informieren

@fenglisch
Copy link
Collaborator Author

#65 (comment) wurde in ein eigenes Issue überführt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants