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

<abbr> Tags werden bei der Übersicht über Suchergebnisse als String serialisiert #40

Open
neradis opened this issue Apr 7, 2014 · 3 comments
Assignees

Comments

@neradis
Copy link
Owner

neradis commented Apr 7, 2014

@andywer

Wenn in der Ergebnisliste der Volltextsuche in den Abkürzungen in den Snippets vorkommen, werden die abbr-Tags zu deren Markierung als Substrings in Textknoten:

Absalom (Absalon) u. Abessalon, <abbr>m.</abbr> indecl., Sohn Davids, Eccl.

Wie aufwendig wäre es denn dafür zu sorgen, dass die Tags tatsächlich als HTML-Tags ins DOM kommen und sie anschließend mit den passenden title-Attributen zu versehen?
Falls das großen Aufwand nach sich ziehen würde, sollten wir zumindest statt dessen im AjaxController dafür sorgen, dass das HTML wieder aus den Snippets rausgefiltert wird.

@andywer
Copy link
Collaborator

andywer commented Apr 8, 2014

Bei mir erscheinen keine -Tags in der Suchergebnis-Kurzbeschreibung. Hab es auch mit "Absalom" versucht.
(Musste die Funktion sofort zur Ergebnisseite zu springen wenn es nur einen Suchtreffer gibt dafür auskommentieren: s. keyword-input.js 57-61)

@neradis
Copy link
Owner Author

neradis commented Apr 10, 2014

@andywer

Hier erstmal kurz zwei Screenshots, damit wir definitv vom gleichen Problem reden:

Ich habe es gerade noch einmal überprüft. Die <abbr>-Tags tauchen sowohl bei d48dc2b als auch bei a2451f8 auf. In beiden Fällen habe ich voher rake assets:clean assets:precompile laufen lassen, um auszuschließen, dass veraltetes CSS/JS genutzt wird.)

Möglicherweise hast du noch Eintragsdaten in deiner DB, bei denen die Abkürzungen noch nicht in dem description-Attribut markiert wurden, sonder nur in htmlDescription. Sofern das nicht in den letzten Tagen schon mal geschah, lass bitte einmal gradle -Denv=test recreateAll laufen.

@andywer
Copy link
Collaborator

andywer commented Apr 10, 2014

Also ich bin auf d48dc2b und bei mir sind allgemein keine <abbr>-Tags drin. Weder in der Ergebnisliste in der Beschreibung, noch auf der Detailseite. Und ja, ich hab gradle -Denv=test recreateAll ausgeführt, den Cache geleert und ich setze ENV=test vor das ./railscmd rails s.

Bin gerade ein bisschen ratlos...

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

No branches or pull requests

3 participants