Mega-Menü -> Qelltext nicht valide

schnakenhuster

Aktives Mitglied
3. Juni 2013
15
0
Hallo allerseits,

ich bin seit einiger Zeit damit beschäftigt, die Validatorfehler aus meinen Projekten zu entfernen. Seit ich folgendes gelesen habe, bin ich auch schon ein ganzes Stück weiter.
PHP:
Dazu in includes/lib/phpQuery/phpquery.class.php Zeile 760
von
     PHP-Code:
      $markup = $this->isXML
     ? $this->document->saveXML()
     : $this->document->saveHTML();  
   nach
     PHP-Code:      $search = array('/<link([^>]*)>/i',  '/<meta([^>]*)>/i', '/<br([^>]*)>/i',  '/<img([^>]*)>/i', '/<base([^>]*)>/i',  '/<input([^>]*)>/i');
$replace = array('<link$1/>',  '<meta$1/>', '<br$1/>', '<img$1/>', '<base$1/>', '<input$1/>');
$markup = $this->isXML
     ? $this->document->saveXML()
     : preg_replace($search, $replace, $this->document->saveHTML());

Nicht elegant und natürlich nicht updatesicher, aber funktional.

Allerdings bindet das Mega-Menü leider mitten im body noch ein paar Stylesheets ein, was einerseits zu Validatorfehlern wie
HTML:
line 162 column 139 - Fehler: document type does not allow element "link" here
fürht, andereseits auch zurecht von Google Page Speed bemängelt wird.
Hat jemand eine Idee, wie sich dieses Problem beheben lässt? Möglichst ohne noch mehr im Core herumzufrickeln....

viele Grüße

Michael
 

casim

Sehr aktives Mitglied
26. Juni 2012
5.934
9
AW: Mega-Menü -> Qelltext nicht valide

einfach ignorieren wäre keine Option für dich?

Glaubst du, du hast 1 Besucher bzw. 1 Kauf mehr deswegen in deinem Shop?
 

schnakenhuster

Aktives Mitglied
3. Juni 2013
15
0
AW: Mega-Menü -> Qelltext nicht valide

naja, also ignorieren ist immer eine Option, aber hier geht es nicht nur um die Validität, sondern ich verschenke auch noch zusätzlich Performanz. Und das die Einfluss auf das Kaufverhalten hat ist unstrittig, oder? Und ich bin der Ansicht, das es möglich sein sollte, ein PlugIn so zu schreiben, dass es validen Quellcode ausgibt....
 

salepix.de

Offizieller Servicepartner
SPBanner
17. Januar 2013
488
49
Monheim / Köln
Firma
SALEPIX GmbH
AW: Mega-Menü -> Qelltext nicht valide

naja, also ignorieren ist immer eine Option, aber hier geht es nicht nur um die Validität, sondern ich verschenke auch noch zusätzlich Performanz. Und das die Einfluss auf das Kaufverhalten hat ist unstrittig, oder? Und ich bin der Ansicht, das es möglich sein sollte, ein PlugIn so zu schreiben, dass es validen Quellcode ausgibt....

@schnakenhuster: hier stimme ich casim voll zu, du übertreibst :)
 

chef1568

Sehr aktives Mitglied
4. Mai 2009
1.265
2
Bayern
AW: Mega-Menü -> Qelltext nicht valide

@casim @salepix,
Natürlich habt ihr Recht und dies ist nur eine Unschönheit - mehr nicht. Es beeinträchtigt weder die Performance merklich, noch das Ranking und schon gar nicht das Kaufverhalten der Besucher.
ABER: es ist ein Schönheitsfehler...und das berechtigt diesen Thread.
So geringwertig dieser "Fehler" ist, so geringwertig ist auch die Behebung (es geht wohl auch schneller als meinen Beitrag zu schreiben).
Aus diesem Grund sollte dieser "Fehler" behoben werden um die potentiellen und derzeitigen Kunden von der 100%igen Qualität zu überzeugen. Dadurch zeigt doch der SP dass er die Wünsche (seien sie noch so geringwertig) der Kunden ernst nimmt.

gruß Feder
 

Kreativkonzentrat

Offizieller JTL-Partner
ZTBanner
26. April 2012
311
80
Dresden
AW: Mega-Menü -> Qelltext nicht valide

Ein definitiv berechtigter Thread!
Ich nehme es mal als Wunsch für kommende Versionen auf (CSS + Javascript im <head> einbinden).

Die Behebung des Fehlers muss leider im Quellcode geschehen, daher gibt es vorerst keinen Quickfix dafür. Derzeit werden die Styles und Scripte vom Template des Menus eingebunden, daher auch ihre Position direkt dort, wo auch das Menu steht.

Danke für den Hinweis und lieben Gruß,
Martin
 

schnakenhuster

Aktives Mitglied
3. Juni 2013
15
0
AW: Mega-Menü -> Qelltext nicht valide

vielen Dank für die Rückmeldung. Sollte ja theoretisch keine allzu große Sache sein....
Im Idealfall sollte sich das CSS dann auch über die integrierte Komprimierung von JTL zusammenfassen lassen. Das reduziert nochmal die Ladezeit und die Server-Requests....
 
Ähnliche Themen
Titel Forum Antworten Datum
Neu S: Plugin Dropdown-Menü für meine Kategorien Plugins für JTL-Shop 10

Ähnliche Themen