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.

Sign up for GitHub

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

Jump to bottom

Issue 70 no details with textfilter #71

Closed
wants to merge 13 commits into from
Closed

Issue 70 no details with textfilter #71

wants to merge 13 commits into from

Conversation

msteudtn
Copy link
Owner

@msteudtn
Addon to show email and telephone-button in the results
db0529c
@msteudtn
Update addon_contacts_in_results.js
ecdb3c7 
related to #65
@msteudtn
Verbesserungen am Add-On
36e4487 
* 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
Verbesserungen und Bugfixes
4d8dd0b 
* 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
Adaptions for use with other addons
5f0d87f 
* Pseudo-CSS-Klasse "showAlwaysIsTrue" für addon_results_textfilter_by_button.js
* Neue Berechnungen um herauszufinden, wie viele Zeilen (CSS class row) in der oberen Liste #resultsShortTable stehen. -> Relevant für Addons, die Zeilen hinzufügen, z.B. addon_contacts_in_results.js
* Eigene Funktion fnFadeIn() und fnFadeOut() deaktiviert. -> Stattdessen nur das CSS verändert, da fnFadeIn() / fnFadeOut() sowohl CSS:display als auch CSS:visibility ändern.
@msteudtn
Adaptions for use with other addons
82b7690 
* Filter für Pseudo-CSS-Klasse "showAlwaysIsTrue" aus "addon_limit_results.js" eingefügt -> Buttons aus anderem Addon nicht filtern.
* Nicht nur CSS:display ändern, sondern auch CSS:visibility. So wird auch Platz eingeräumt.
@msteudtn
Add files via upload
3d355a6
@msteudtn
Copy link
Owner Author

@fenglisch
Copy link
Collaborator

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
Copy link
Owner Author

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
Copy link
Collaborator

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
Testzeile, Parteiname im Mail-Betreff
1c45177 
* Testzeile ohne Funktion für andere Addons
* Parteiname wird in den E-Mailbetreff übernommen
* #65
@msteudtn msteudtn mentioned this pull request Aug 1, 2022
Closed
@msteudtn
Copy link
Owner Author

msteudtn commented Aug 1, 2022

#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
Copy link
Collaborator

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
Update addon_contacts_in_results.js
2c42842 
* Formatierungen / Klassen entfernt für Update in OUTPUT.JS ("Klammer" um Zeile)
* Ergebnisse werden in E-Mail angehängt
@msteudtn
Update addon_limit_results.js
e179e30 
* Anpassungen für "neue Klammer"-Zeile in OUTPUT.JS
@msteudtn
Update output.js
d29f675 
* neue "Klammer"-Zeile
* Wenn ein Addon (z.B. addon_contacts_in_results.js) eine neue Zeile unter die Zeile #resultsShortParty einfügt, bleiben die Zebrastreifen aus der Klasse ".mow-row-striped" in der richtigen Reihenfolge.
@msteudtn
Add files via upload
b42a656
@msteudtn
Merge branch 'issue-65-addon-contact-buttons' into issue-70-no-detail…
8d43783 
…s-with-textfilter
@msteudtn
Copy link
Owner Author

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

@msteudtn msteudtn closed this Aug 27, 2022
@msteudtn msteudtn deleted the issue-70-no-details-with-textfilter branch February 3, 2023 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@fenglisch fenglisch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@msteudtn @fenglisch