Projekt

Allgemein

Profil

Aktionen

GX-Bug #47519

geschlossen

Megadropdown Rel-Attribut ersetzen

Von Timo Backhaus vor mehr als 8 Jahren hinzugefügt. Vor mehr als 3 Jahren aktualisiert.

Status:
Erledigt
Priorität:
Hoch
Zugewiesen an:
Mirko Janssen
Kategorie:
Frontend
Zielversion:
% erledigt:

100%

Geschätzter Aufwand:
Steps to reproduce:
Betroffene Versionen:
3.0.0.0
Release Notes Langtext:

Das Megadropdown verwendet nun nicht mehr das rel-Attribut und weitere HTML-Fehler wurden aus dem Honeygrid-Template entfernt

Tags:
Fix

Beschreibung

Im Megadropdown wird bei den Links das rel Attribut missbraucht.

Lösung (MegadropdownHandler.js):
var t_categories_id = $(this).parent().attr('id').replace('megadropdown_top_link_id_', '');

Dadurch wird nicht das rel Attribut ausgelesen, sondern die ID des li Elementes, welches ebenfalls die Kategorie ID beinhaltet.


Dateien

megadropdown_1.png (44,8 KB) megadropdown_1.png Mirko Janssen, 27.03.2017 13:27
megadropdown_2.png (177 KB) megadropdown_2.png Mirko Janssen, 27.03.2017 13:27

Aktionen #1

Von Till Tepelmann vor fast 8 Jahren aktualisiert

  • Priorität wurde von Normal zu Hoch geändert
Aktionen #2

Von Daniel Wu vor mehr als 7 Jahren aktualisiert

Evtl. bereits von Torben gelöst

Aktionen #3

Von Torben Wark vor mehr als 7 Jahren aktualisiert

Das habe ich in der Tat schon mitgelöst, zusammen mit dem Schwall an Änderungen, die ich zum Thema HTML-Validierung gemacht habe. Das ist allerdings ein wenig umfangreicher, sodass sich hier die Frage stellt, ob das nicht geeigneter für einen künftigen Feature-Release ist.

Aktionen #4

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Tags wurde von Fix zu Fix, später geändert
  • Status wurde von Gemeldet zu Feedback geändert
  • Zugewiesen an wurde auf Torben Wark gesetzt
Aktionen #5

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Status wurde von Feedback zu In Bearbeitung geändert
Aktionen #6

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Status wurde von In Bearbeitung zu Testing geändert
  • Zugewiesen an wurde von Torben Wark zu Core-Entwickler geändert
  • Release Notes Langtext aktualisiert (Vergleich)

Lösung für EyeCandy von Timo übernommen. In Honeygrid wurde bereits eine andere Lösung entwickelt, als an der HTML-Validierung gearbeitet wurden ist.

Aktionen #7

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Zielversion wurde von 133 zu 3.5.1.0 beta1 geändert
Aktionen #8

Von Daniel Würdemann vor mehr als 7 Jahren aktualisiert

  • Zugewiesen an wurde von Core-Entwickler zu Daniel Würdemann geändert
Aktionen #9

Von Daniel Würdemann vor mehr als 7 Jahren aktualisiert

  • Status wurde von Testing zu In Prüfung geändert
  • Zugewiesen an wurde von Daniel Würdemann zu Torben Wark geändert

Bitte folgende Dateien (Zeilen) prüfen

src/templates/EyeCandy/boxes/box_categories_left.html : 8
src/templates/EyeCandy/boxes/box_categories_top.html : 8
src/templates/EyeCandy/javascript/MegadropdownHandler.js : 66

Aktionen #10

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Status wurde von In Prüfung zu In Bearbeitung geändert
Aktionen #11

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Status wurde von In Bearbeitung zu Testing geändert
  • Zugewiesen an wurde von Torben Wark zu Core-Entwickler geändert
Aktionen #12

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Release Notes Langtext aktualisiert (Vergleich)
Aktionen #13

Von Mirko Janssen vor mehr als 7 Jahren aktualisiert

  • Zugewiesen an wurde von Core-Entwickler zu Mirko Janssen geändert
Aktionen #14

Von Mirko Janssen vor mehr als 7 Jahren aktualisiert

Beim Mouseover erhalte ich einen JS Fehler -> siehe Bilder.

Aktionen #15

Von Torben Wark vor mehr als 7 Jahren aktualisiert

  • Zugewiesen an wurde von Torben Wark zu Core-Entwickler geändert
Aktionen #16

Von Mirko Janssen vor mehr als 7 Jahren aktualisiert

  • Zugewiesen an wurde von Core-Entwickler zu Mirko Janssen geändert
Aktionen #17

Von Mirko Janssen vor mehr als 7 Jahren aktualisiert

  • Status wurde von Testing zu Erledigt geändert
  • % erledigt wurde von 0 zu 100 geändert
Aktionen #18

Von Moritz Bunjes vor mehr als 3 Jahren aktualisiert

  • Tags wurde von Fix, später zu Fix geändert
Aktionen

Auch abrufbar als: Atom PDF