Auf das id Attribute verzichten und einheitliche css-Selektoren

[tbaddade]

Momentan werden Ids mit gleichen Namen ausgegeben. Damit ist das HTML invalide. Ich würde vorschlagen das YForm gar keine Ids mehr mit ausgibt.

CSS-Selektoren werden aktuell mal mit _mal mit -getrennt. Außerdem variiert die Schreibweise: formlabel, form-label, yform-ycom, yformycom, yform_ycom

[dergel]

lass uns die schreibweise vereinheitlichen - was schlägst du vor ? form-label und yform-ycom-.. (wäre dann auch eher teil von ycom) ..
Wir brauchen an einigen Stellen die ID, gerade bei Backend dingen, relationen oder select krams. dennoch kann man die ID mitlerweile an einigen Stellen bereit über die attributes austauschen.

[tbaddade]

was schlägst du vor ?

- als Trenner. Wäre konform zum Core. Bei der Ycom wäre dann wohl yform-ycom- als Prefix logisch.

Wir brauchen an einigen Stellen die ID, gerade bei Backend dingen, relationen oder select krams.

Dann würde ich die Id nur noch dort ausgeben lassen, wo sie tatsächlich benötigt wird. Für js-Kram würde ich sogar wie im Core auf das Prefix rex-js- oder besser yform-js- im Css-Klassennamen gehen (Vorteil wäre, dass man im Code gleich erkennt das Js ausgelöst wird. Diese Klassen werden dann im CSS selbst nirgends notiert sondern stehen nur Js zur Verfügung) oder via data-mach="was"Attribut.

[dergel]

Kannst du die Beispiele raussuchen bei denen der "-" noch falsch ist ?

[christophboecker]

ID's sind ja nicht grundsätzlich schlecht. Bin auch grade darauf gelaufen, dass in YForm-Formularen das Suchfeld und das Eingabefeld die gleiche ID "rex-yform" haben. Jetzt fehlt mir der eindeutige Qualifier (hier für CSS). Ich helfe mir mit "#rex-js-page-main > .row > div:first-child" um auf das Suchfeld zu kommen. Aber schön geht anders.

Für den konreten Fall ist Lösung ja einfach. In der searchform-Klasse einfach die Default-ID ersetzen.

class rex_yform_manager_searchform extends rex_yform
{

    public function __construct()
    {
        parent::__construct();
        $this->objparams['form_wrap_id'] = 'rex-yform-search';
    }

Braucht es dafür einen Pull-Request oder reicht das so?

Grüße
Chris

[dergel]

ids müssten auf jeden fall raus.. gerade auch wegen inline relationen, bei denen dann identische id zuhauf auftauchen würden.

[dergel]

müssten entscheiden wie es am sinnvollsten ist,
-div wrapper
-- form
--- element wrapper
---- element
.
.
.
.

[alxndr-w]

wurde das nicht mit #824 ganz oder teilweise gelöst?