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

Search Highlighter zeichnet auch ... aus #186

Open
alexwenz opened this issue Feb 21, 2019 · 12 comments · Fixed by #187 or #190
Open

Search Highlighter zeichnet auch ... aus #186

alexwenz opened this issue Feb 21, 2019 · 12 comments · Fixed by #187 or #190
Assignees

Comments

@alexwenz
Copy link
Member

Die Suche nach Winter und Übergabe des Suchbegriffes via ?search_highlighter=winter erzeugt folgenden Quellcode:

<img src="/images/overlay/<span class='searched-it'>winter</span>1.jpg" alt="" width="60" height="16">

@tyrant88
Copy link
Member

Es liegt an dem HTML-tag im Attribut, da matched der Reguläre Audruck wieder, obwohl er nicht sollte.
gestern 4 RegEx Ansätze probiert.
Muss ich wohl mit einer ganz anderen Logik lösen..

@tyrant88 tyrant88 self-assigned this Feb 22, 2019
@alxndr-w
Copy link
Member

Kannst du es in 2 Etappen lösen? Erst ersetzen, dann dort filtern, wo sie zu viel sind? Als Lösungsansatz

@tyrant88
Copy link
Member

Ja so ähnlich ist der Gedanke.

@tyrant88
Copy link
Member

Nächster Versuch. Alex meint es geht jetzt Alex. Willst du auch noch mal testen?

@alxndr-w
Copy link
Member

Wenn Alex das sagt, dann vertraue ich darauf. Alex' haben generell einen guten Ruf ;)

@tyrant88
Copy link
Member

Ja, wenn Kevin das gesagt hätte würde ich nicht soviel drauf geben... :-)

@danspringer
Copy link
Member

Darf ich mich hier noch einklinken?
Der Highlighter zeichnet auch Treffer z.B. in einer <textarea> aus, die bereits mit Werten aus einem Artikel vorbelegt ist, was zu HTML in den Inputs eines Formulars führt.

@alxndr-w alxndr-w reopened this Mar 11, 2019
@alxndr-w
Copy link
Member

alxndr-w commented Mar 11, 2019

<button></button> dürfte auch so ein Kandidat dafür sein.

edit:

<button class="btn btn-primary" type="submit" name="FORM[anfrage-formular][19]" id="yform-anfrage-formular-field-19-abschicken" value="Abschicken">Ab<span class="search-it-highlighter">schick</span>en</button>

jop.

@tyrant88 tyrant88 changed the title Search Highlighter zeichnet auch src-Attribute von Bildern aus Search Highlighter zeichnet auch ... aus Mar 11, 2019
@tyrant88
Copy link
Member

Hab leider gerade null Zeit

@xong
Copy link
Member

xong commented Mar 11, 2019

Als Anregung: Ich kann man mir vorstellen, dass man ähnlich wie im Plaintext-Plugin mit dem PHP Simple HTML DOM Parser arbeitet. Dort hatte ich für RexSearch eine remove-Methode implementiert. Wahrscheinlich kann man ähnlich auch eine "highlight-replace"-Methode schreiben, die nur Inhalte bestimmter Tags parst und gefundene Suchbegriffe hervorhebt.

Alternativ könnte man auch das Highlighting mit Javascript umsetzen und z. B. mark.js verwenden. Das erscheint mir die beste und performanteste Lösung zu sein.

@alexwenz
Copy link
Member Author

alexwenz commented Mar 11, 2019

Javascript wäre sicher eine gute Lösung, hätte aber den Nachteil, dass man das im Frontend einbinden und damit die schöne "seach_it macht alles autark"-Situation aufgeben müsste ...

@xong
Copy link
Member

xong commented Mar 12, 2019

Der Ansatz war eigentlich immer, dass die Ausgabe ins Modul gehört und das Addon vielfältige Funktionen und Optionen anbietet, um dabei zu unterstützen. Das "Search Highlighter"-Plugin wurde übrigens auch nie von mir programmiert, weil es nicht ins Konzept gepasst hat. Es kam quasi von außen.

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

Successfully merging a pull request may close this issue.

5 participants