PRODUCTS_SHORT_DESCRIPTION in p-tags validiert nicht

[noRiddle (revilonetz)]

Hallo liebe Community.
Nicht, daß ich ein Validierungsfanatiker bin aber ich habe doch eine zwingende Verbesserung vorzuschlagen.

Version 1.05 SP1a

Alle Short Descriptions für die Artikel sind in p-tags eingefasst.
Dies validiert nicht nur nicht sondern es ist unzulässiges XHTML mit welchem die Browser nur etwas anfangen können weil sie "tolerant" sind.
Die Short Description ist folgendermassen implementiert:

Code: PHP  [Auswählen]

<p>{$module_data.PRODUCTS_SHORT_DESCRIPTION}</p>

Da, wenn man eine Kurzbeschreibung für einen Artikel verfasst, paragraphes, also p-tags, vorkommen werden (insbesondere wenn man den Editor benutzt),
entstehen p-tags innerhalb von p-tags, was nicht zulässig ist.
Firefox generiert daraus beispielsweise leere p-tags um den Kurzbeschreibungstext dann in seinen extra p-tags darzustellen (siehe z.B. mit Hilfe von Firebug).

betroffen sind folgende Dateien:
templates/YOUR_TEMPLATE/modules/

• also_purchased.html
• cross_selling.html
• new_products.html
• new_products_default.html
• new_products_overview.html
• reverse_cross_selling.html
• product_listing/product_listing.html

Ich empfehle dringend die p-tags durch div-tags zu ersetzen, welchen man bei der Gelegenheit auch gleich eine CSS Klasse vergeben könnte, da der Text dann über CSS ansprechbar wäre.
Z.B.:

Code: PHP  [Auswählen]

<div class="short_desc">{$module_data.PRODUCTS_SHORT_DESCRIPTION}</div>

Die Änderung ist schnell gemacht, produziert validen Code und verwirrt auch zukünftige standardkonforme und vielleicht nicht so tolerante, Browser nicht.

Have a nice day,
noRiddle

Linkback: https://www.modified-shop.org/forum/index.php?topic=17244.0

[franky_n]

Hallo noRiddle,

 

Viele Grüße

Franky

[Tomcraft]

[...]

Code: PHP  [Auswählen]

<p>{$module_data.PRODUCTS_SHORT_DESCRIPTION}</p>

[...]

Im Trunk ist das bereits ersetzt durch:

Code: PHP  [Auswählen]

<br />{$module_data.PRODUCTS_SHORT_DESCRIPTION}

Grüße

Torsten

[noRiddle (revilonetz)]

Sollte man nicht um der Semantik Willen die description in div-tags einfassen ?

Gruß,
noriddle

[hedon02]

Semantik und "div" schliesst sich doch fast aus.  

[noRiddle (revilonetz)]

 

[hedon02]

okay Korrektur... schliesst einander immer aus. 

Gruesse

[Matt]

Semantik und "div" schliesst sich doch fast aus.  

Du kannst es ja in ein <section> packen und den Doctype ändern.