html/text ratio

Krisis schrieb:
Weil die letzte Seite beim Laden im letzten Topic weiß bleibt und ein Antworten nicht mehr möglich ist, mache ich einen neuen Thread zu dem Thema auf.

Ja, manchmal führt irgendein Code-Zitat-Mix zu interessanten Ergebnissen.

Krisis schrieb:
Ich sehe exemplarisch das hier. Welchen Bereich rund um die ID kann ich entfernen?

Das:
<?php echo $header_left; ?>"><h3 class="leftBoxHeading
<?php echo $header_right; ?>"
id="<?php echo str_replace
(\'_\', \'-\', $box_id) . \'Heading\'; ?>">
Wird zu:
<?php echo $header_left; ?>"><h3 class="leftBoxHeading
<?php echo $header_right; ?>">

Krisis schrieb:
Ich habe diesen Suchstring gefunden. Ich weiss aber nicht, wie ich das auslagern kann. Der Suchstring ist nur in einer php Datei vorhanden, aber 10x im Quelltext. Das sollte einiges an Spaghetti verputzen

return $parameters . \'style="position:relative"
onmouseover="showtrail(\' .
"\'$products_image_zoom\',\'$alt\',$width,$height,
$zoomwidth,$zoomheight,this," .
$this->zoom[\'startx\'].\',\'.$this->zoom[\'starty\']
.\',\'.$this->zoom[\'width\'].\',\'.$this->zoom[\'height\'].\');"
onmouseout="hidetrail();" \';

Das wird schön, wenn das mal korrekt ausgegeben wird. Ohne die Event Handler ist der Quelltext nur noch halb so groß.
style="position:relative" müsste aus dem Code raus und "position:relative" in die CSS-Datei übernommen werden. Schau im fertigen HTML-Code, welche CSS-Klasse das HTML-Element hat und ergänze es dort.

Guten Morgen!

Check!
Die ids aus den Sideboxen sind draussen.

To Do
Die id des centerboxwrappers soll noch raus.

position:relative liegt in folgenden Klassengruppen:
- new, centeredcontent, back, col33
- special, centeredcontent, back, col33
- sideboxcontent, centeredcontent

Sind es die drei Klassen new, special und sideboxcontent, in die position:relative eingetragen werden soll?

Wird aus
return $parameters . \'style="position:relative"
onmouseover="showtrail(\' .
"\'$products_image_zoom\',\'$alt\',$width,$height,
$zoomwidth,$zoomheight,this," .
$this->zoom[\'startx\'].\',\'.$this->zoom[\'starty\']
.\',\'.$this->zoom[\'width\'].\',\'.$this->zoom[\'height\'].\');"
onmouseout="hidetrail();" \';

das hier?
return $parameters . \'
onmouseover="showtrail(\' .
"\'$products_image_zoom\',\'$alt\',$width,$height,
$zoomwidth,$zoomheight,this," .
$this->zoom[\'startx\'].\',\'.$this->zoom[\'starty\']
.\',\'.$this->zoom[\'width\'].\',\'.$this->zoom[\'height\'].\');"
onmouseout="hidetrail();" \';

Krisis schrieb:
Sind es die drei Klassen new, special und sideboxcontent, in die position:relative eingetragen werden soll?

Ich würde eine nehmen, die alle Kandidaten gemeinsam haben.

Krisis schrieb:
Wird aus
return $parameters . \'style="position:relative"
onmouseover="showtrail(\' .
"\'$products_image_zoom\',\'$alt\',$width,$height,
$zoomwidth,$zoomheight,this," .
$this->zoom[\'startx\'].\',\'.$this->zoom[\'starty\']
.\',\'.$this->zoom[\'width\'].\',\'.$this->zoom[\'height\'].\');"
onmouseout="hidetrail();" \';

das hier?
return $parameters . \'
onmouseover="showtrail(\' .
"\'$products_image_zoom\',\'$alt\',$width,$height,
$zoomwidth,$zoomheight,this," .
$this->zoom[\'startx\'].\',\'.$this->zoom[\'starty\']
.\',\'.$this->zoom[\'width\'].\',\'.$this->zoom[\'height\'].\');"
onmouseout="hidetrail();" \';

Genau. Der Teil ist etwas unübersichtlich, aber daran arbeiten wir noch.

Ich glaub es hat geklappt. Es hat jetzt zuletzt, muss ich zugeben echt mit Glück und Ausprobieren zu tun gehabt.
Die Klassen in der stylesheet waren mehrfach vorhanden und ich wusste nicht genau in welche der gleichnamigen Klassen ich den Wert eintragen sollte. Ich hatte die stylesheet.css in der Vergangenheit auch schon unbekümmerter editiert.
Jetzt war ich aber konfrontiert mit durch Kommatas abgegrenzten Klassen und IDs, die hintereinander ohne geschweifte Klammer eingetragen waren uvm.

Es sind noch mehrere Styles vorhanden, die ausgelagert werden müssten.
Das betrifft die Suche und hauptsächlich die beiden Spalten am Rand. Die Zeilen zu finden und zu korrigieren sollte mittlerweile eine einfache Übung sein, denke ich.

Als übersichtlichere Übung zur Vorbereitung des Umbaus der Showtrail-Geschichte könntest Du onfocus und onblur des Such-Eingabefeldes in eine externe JavaScript-Datei verlagern. Das sollte eigentlich möglich sein. "addEventListener" wäre da das Stichwort, Tutorial suchen und diese Funktion auslagern, so dass das Feld am Ende wie gewohnt funktioniert, aber so aussieht:
<input type="text" name="keyword" size="6" maxlength="42" value="Suche nach..." />

Was macht der div eigentlich:

Ich sehe nirgendwo eine dotted-line...

Und dann wären da noch die IDs auf praktisch jedem Container.
Da müsste man für jede prüfen, ob CSS oder JavaScript sie verwenden und wenn nicht, sie rausnehmen. Z.B. bei

Matthes schrieb:

Es sind noch mehrere Styles vorhanden, die ausgelagert werden müssten.
Das betrifft die Suche und hauptsächlich die beiden Spalten am Rand. Die Zeilen zu finden und zu korrigieren sollte mittlerweile eine einfache Übung sein, denke ich.

Hallo Matthes,
bin dran!

Matthes schrieb:

Als übersichtlichere Übung zur Vorbereitung des Umbaus der Showtrail-Geschichte könntest Du onfocus und onblur des Such-Eingabefeldes in eine externe JavaScript-Datei verlagern. Das sollte eigentlich möglich sein. "addEventListener" wäre da das Stichwort, Tutorial suchen und diese Funktion auslagern, so dass das Feld am Ende wie gewohnt funktioniert, aber so aussieht:
<input type="text" name="keyword" size="6" maxlength="42" value="Suche nach..." />

Das muss ich nach hinten schieben.

Matthes schrieb:

Was macht der div eigentlich:

Ich sehe nirgendwo eine dotted-line...

Check!
Ist weg!

Matthes schrieb:

Und dann wären da noch die IDs auf praktisch jedem Container.
Da müsste man für jede prüfen, ob CSS oder JavaScript sie verwenden und wenn nicht, sie rausnehmen. Z.B. bei

Beim body ganz besonders:
<body id="indexHomeBody">
Dem Body eine ID zu geben ist Unsinn, der ist ohnehin einmalig.

Bin dran!

Matthes schrieb:

Solche Konstrukte sollte man auch prüfen:

Erstens ist die Benennung sehr fehleranfällig, bis auf einen Buchstaben ist es identisch, zweitens braucht man sicher nicht drei mögliche Selektoren, um einen div und den direkten Kind-div zu gestalten.

Ich habe gerade aber keine Ahnung, wie ich das angehen soll.
Da war ja bereits das Auffinden der dynamisch erzeugten sidebox IDs schon ganz schön schwierig für mich.

Ich habe den Hovereffekt deaktiviert.

Dann nimm auch noch das JavaScript aus dem Quelltext, jscript_imagehover.js

Ganz unten kann der clearBoth-div wahrscheinlich noch weg, der ist das letzte Element, was soll da noch groß gecleart werden?

Und das Google-Analytics-Script wird direkt vor den schließenden body verschoben, so dass die schließenden divs davor stehen.

Dann bleiben noch die restlichen style-Attribute. Wenn die weg sind, ist nicht mehr viel übrig, auf Anhieb sehe ich gerade nichts. Alles was man jetzt noch machen könnte, würde auf größere Umbauaktionen hinauslaufen.

Matthes schrieb:

Dann nimm auch noch das JavaScript aus dem Quelltext, jscript_imagehover.js

Check! Ist raus.
Ich musste das imagehover.js komplett aus dem Ordner löschen weil es bei Zen-Cart automatisch geladen wird.
Da befindet sich noch eine jquery Datei. Komisch dass diese nicht im Quelltext erscheint.

Matthes schrieb:

Ganz unten kann der clearBoth-div wahrscheinlich noch weg, der ist das letzte Element, was soll da noch groß gecleart werden?

Check!

Matthes schrieb:

Und das Google-Analytics-Script wird direkt vor den schließenden body verschoben, so dass die schließenden divs davor stehen.

Check!

Matthes schrieb:

Dann bleiben noch die restlichen style-Attribute. Wenn die weg sind, ist nicht mehr viel übrig, auf Anhieb sehe ich gerade nichts. Alles was man jetzt noch machen könnte, würde auf größere Umbauaktionen hinauslaufen.

Bin dran!

Zur body ID habe ich nur den folgenden Eintrag gefunden:
$body_id = ($this_is_home_page) ? \'indexHome\' :
str_replace(\'_\', \'\', $_GET[\'main_page\']);

und
<body id="<?php echo $body_id . \'Body\'; ?>"<?php if($zv_onload !=\'\') echo \' onload="\'.$zv_onload.\'"\'; ?>>

In der stylesheet gibt es keinen String, der indexhomebody enthält.

Die verschiedenen templates haben auszugsweise folgenden Inhalt:
$content = \'\';
$content .= \'

Krisis schrieb:
In der stylesheet gibt es keinen String, der indexhomebody enthält.

Einfach im Template <body> daraus machen und abhaken.

Krisis schrieb:
Kann ich daraus folgenden String machen?
$content = \'\';
$content .= \'

\';

Nicht ganz, das id=" vor class sollte noch raus.

Krisis schrieb:
Ich weiss nicht, warum für jede Sidebox extra IDs vergeben worden sind.
Gibt es eine Möglichkeit zu überprüfen, ob die IDs an einer bestimmten Stelle tatsächlich benötigt werden?