w3c Konformität (korrekte Tags, Attribute und CSS-Deklarationen)

[jkatzorke]

Hallo zusammen,

eins vorweg: es handelt sich nicht um Bugs im engeren Sinne, allerdings finde ich es dennoch wichtig, es anzusprechen.

Bei der Validierung eines Shops fiel mir auf, daß modified eCommerce Shopsoftware noch einige Fehler des ursprünglichen xt-commerce enthält:

CSS-Anweisung "cursor:hand"...
Auch wenn Browser den Wert "hand" teilweise noch unterstützen, ist er meines Wissens nicht W3C-konform.
Kommt zwar nicht oft vor, aber mancherorts steht es noch statt "cursor:pointer".

Attribut "onClick"...
XHTML ist case-sensitiv. Leider geistert auch dieses Attribut noch im Quelltext herum. Genauer gesagt 372 Mal.

Gruß Johannes



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

[Tomcraft]

"cursor:hand" sollte eigentlich gegen "cursor:pointer" ausgetauscht sein.

Die anderen Fehler werde ich demnächst korrigieren, danke für den Hinweis!

Grüße

Torsten

[jkatzorke]

modified eCommerce Shopsoftware 1.04:

"cursor:hand":
/admin/includes/modules/fckeditor/editor/dialog/common/fck_dialog_common.js
Zeile 61

admin/includes/modules/new_category.php
Zeile 375

admin/module_newsletter.php
Zeilen 518 + 527

admin/orders.php
Zeile 257

includes/modules/product_info.php
Zeile 117

includes/modules/products_media.php
Zeile 78

"onClick":
popup_image.php
Zeile 105

includes/modules/order_total/ot_gv.php
Zeile 50

includes/modules/products_media.php
Zeile 78

und einige Dateien im Backend...

Backend ist ja nicht so wichtig, auch wenn es natürlich Sinn macht, dort auch auszubessern

[Tomcraft]

Sage mal... könntest du vielleicht ein Patch-Paket bereitstellen?

Grüße

Torsten

[jkatzorke]

Ein Patch-Paket mit allen veränderten Dateien?

Wenn Du mir verrätst, wie das am einfachsten geht

Dachte, es geht schneller, wenn jemand mit SVN-Zugriff das Projekt in Eclipse lädt und über die Suchen & Ersetzen Funktion die betroffenen Dateien korrigiert...

Ich arbeite sonst nicht mit Patch-Paketen

[Tomcraft]

Achso... ich dachte da einfach an ein ZIP-Paket, wo einfach die geänderten Dateien drin sind.

Dann würde ich das nochmal mit einem Merger anschauen und dann integrieren.

Grüße

Torsten

[speedy]

Dachte, es geht schneller, wenn jemand mit SVN-Zugriff das Projekt in Eclipse lädt und über die Suchen & Ersetzen Funktion die betroffenen Dateien korrigiert...

Das denke ich mir auch öfters, das ginge so doch viel schneller. Noch dazu ist dieser Weg nicht so fehlerträchtig.

Mit Leerzeichen, also

cursor: hand

gäbe es auch noch einiges, aber alles die Admin betreffend.

[Tomcraft]

Fehler behoben in r961 & r962.

Grüße

Torsten

[speedy]

Danke dir Torsten - da hast dich ja wieder ausgetobt, 77 modifizierte Dateien - hehe.
Jetzt stell dir das als ZIP vor *grusel*

Aber das böse unkontrollierte dateiübergreifende suchen und ersetzen hat wieder zugeschlagen

z.B.
admin/includes/modules/fckeditor/editor/dialog/common/fck_dialog_common.css

Code: PHP  [Auswählen]

80      .BtnOver
81      {
82              border: outset 1px;
83              cursor: pointer;
84              cursor: pointer;
85      }
 

Also die Änderungen beim fckeditor müsste man auf alle Fälle noch einmal kontrollieren, ob es ähnliche Stellen öfters gibt

[Tomcraft]

Danke für den Hinweis! Da wollte ich eh nochmal drüber schauen, denn ich kenne genau diese Tücken.

Geht aber trotzdem schneller das erstmal "quick & dirty" ein zu pflegen und danach auszubessern, später im fertigen Paket bekommt ihr davon ja nichts mehr mit.

Grüße

Torsten

[DokuMan]

Wie gut, dass "Speedy" immer ein "schnelles" Auge auf sowas hat!

[Tomcraft]

So, hier die Fixes in r963 & r964.

Bitte nochmal rüber schauen.

Dort wo im FCKeditor nur ein cursor:hand drin war ohne pointer Anweisung davor, habe ich gegen cursor:pointer ersetzt.

Danke nochmal speedy!

Grüße

Torsten

[speedy]

Sieht gut aus, waren doch ein paar Stellen.
Sollte noch etwas sein, komm ich spätestens beim nächsten Release damit an
Das fällt dann beim 1:1 Dateivergleich auf, doch den Spaß gönne ich mir nicht so oft - hihi.

[Tomcraft]

Ich habe ja auch nochmal gründlich rüber geschaut, also waren da schonmal vier Augen drauf gerichtet und der Trunk Demoshop funktioniert ja auch noch, das ist immer ein gutes Zeichen. *gg*

Grüße

Torsten

[jkatzorke]

Danke dir Torsten - da hast dich ja wieder ausgetobt, 77 modifizierte Dateien - hehe.
Jetzt stell dir das als ZIP vor *grusel*
[...]

Eben... Da wäre es fast einfacher gewesen, wenn ich direkt den ganzen Shop gezippt hätte

Aber für die Zukunft: Gibt es ein SVN mit Schreibrechten? Gibt da noch einiges zum Anpassen... Z. B. die Tabellen bei den Popups... Die sollten raus bzw. würde ich dafür wieder Templates vorsehen, wie bei anderen Inhalten... Ist dann nur umständlich, wenn ich nach einem Update wieder Core-Hacks vornehmen muss...
Andererseits möchte ich Torsten auch nicht mit ZIPs bombardieren, die er jedes Mal mit WinMerge analysieren muss...

Gruß Johannes