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

Konflikt zwischen Kontakt-Addon, Filter-Addon und Limit-Results-Addon: Anzeige der Kontakt-Buttons #73

Open
fenglisch opened this issue Sep 4, 2022 · 1 comment
Labels
bug prio3 low priority

Comments

@fenglisch
Copy link
Collaborator

fenglisch commented Sep 4, 2022

Übernommen aus #65

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 fenglisch added bug prio3 low priority labels Sep 4, 2022
@msteudtn
Copy link
Owner

msteudtn commented Feb 3, 2023

Auf der Ergebnis-Zeile liegt ein Event-Listener mit einem toggle()-Event.

Je nachdem, was aktiv ist, wird die jeweilige "Unterzeile" (Beschreibung, Kontakt-Buttons, Vereins-Angebote) eingeblendet oder ausgeblendet.

Um das zu lösen, müsste man (irgendwo?) eine umfangreiche Prüfung durchführen.

WENN Ergebniszeile sichtbar UND (1 / x Unterzeilen sichtbar) DANN blende restliche Unterzeilen ein.

Ich glaube, dass hier Kosten und Nutzen in keinem Verhältnis stehen. :(

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

No branches or pull requests

2 participants