Support » Plugin: Shariff Wrapper » W3C

  • Hi,

    ich habe bei der Mailform (/?view=mail) W3C Fehler entdeckt. Auch wenn es für die Funktion nicht wichtig ist, könnte das im nächsten Update behoben werden:)

    Error: Element legend not allowed as child of element div in this context. (Suppressing further errors from this subtree.)
    
    From line 289, column 47; to line 289, column 54
    
    headline"><legend>Diesen
    Error: The for attribute of the label element must refer to a non-hidden form control.
    
    From line 297, column 11; to line 297, column 28
    
    							<p><label for="name">Name d
    The for attribute of the label element must refer to a non-hidden form control.
    
    From line 299, column 11; to line 299, column 36
    
    							<p><label for="mail_comment">Zusatz

    Gruss Harry

Viewing 10 replies - 1 through 10 (of 10 total)
  • Plugin Author Jan-Peter

    (@starguide)

    Thanks Harry, as always!

    I will fix it with the next release.

    Cheers
    JP

    gerne doch;)

    Ich hab da noch was:)
    Bei bestimmten Diensten, wie zb. Reddit, wären Scrollbalken bei der aktuellen Grösse des Popups hilfreich. Man erreicht sonst nur via Pfeiltasten den Button zum Abesenden oder auch zum Login. Ein “scrollbar=yes,” nach “location=no,” in der shariff-popup.min.js wäre da eine Idee:)

    Plugin Author Jan-Peter

    (@starguide)

    Ich weiß, warum ich Popups nicht mag. 😀 Aber irgendwie mögen die Leute die Dinger oder fragen einfach zu oft danach. Ich probier mal rum!

    Oder einfach das Popup grösser machen;) Oder die Grösse im Backend anpassbar machen? Oder alternativ als Option im Backend angeben ob Scrollbalken gewünscht sind. Zusätzlich dann noch bei dem jeweiligen “Dienst” (wie bei mail, infolink, etc) ob im neuen Tab oder Popup geöffnet werden soll?

    Nein Spass, ich weiss was du meinst und da kannst du es nicht jedem Recht machen und müsstest dann tatsächlich alles übers Backend anpassbar machen….
    Tatsächlich sind hier Popups aber sinnvoll

    Plugin Author 3UU

    (@3uu)

    Hm, zu diesem “W3C-Fehler” stehe ich jetzt vielleicht auf dem Schlacuh, aber die Elemente sind imho ordentlich im fieldset-Block drin. Und werden ja auch genutzt, um nicht plausible Eingaben zu markieren. Und Browser-Unterstuetzung ist laut http://www.w3schools.com/tags/tag_fieldset.asp auch ganz gut. Ich waere da jetzt eigentlich nicht sonderlich motiviert, diese Funktionalitaet rauszuwerfen, nur weil der W3C parser damit nicht klar kommt.

    Plugin Author Jan-Peter

    (@starguide)

    Die W3C-Fehler sind kein Thema:

    1) <legend> muss das ERSTE Element sein, das ist aber ein <div>. Ergo einfach umbauen.

    2) Das input-Feld heißt “sender” und nicht “name”, blöder Benennungsfehler.

    3) Beim input-Feld fehlt die id.

    Also alles lösbar.

    Gruß
    JP

    Hi und danke:)
    Ich hatte im code zum testen aus dem <legend> einfach ein <span> gemacht. Das es das erste Element sein muss sagt ja der Validator soweit schon:)
    Dann setze ich das bei mir schonmal so um.

    Ich habe noch eine Frage zu dem letzten Inputfeld (Zeile 1090 in der shariff.php). Sollte da nicht ein Placeholder stehen? Der User weiss so ja nicht, dass er dort eine URL eintragen könnte. Wobei zum teilen ist das ja eigentlich so gar nicht nötig noch eine URL mit zu verschicken. Oder erfüllt das Feld noch einen anderen Zweck? Ich habe es bei mir momentan auskommentiert.

    Dann habe ich festgestellt, dass das “X” zum Schliessen des Formulars ein wenig “runder”(ohne Kante im FF) erscheint, wenn in Zeile 1079 (und auch 1283) das width nicht 20 sondern 21 wäre.

    Gruss Harry

    Plugin Author Jan-Peter

    (@starguide)

    Das url-Feld ist nur nen Honeypot für Spam-Bots. Der User sieht das gar nicht.

    Das mit dem X kann ich gerne aufnehmen.

    Gruß
    JP

    Achso, dann ist ja gut:)
    Ist das so grundsätzlich mit CSS versteckt? Denn ich hab ja meine eigene CSS, da wird das dann wohl fehlen;) Denn bei mir sieht man es:)

    Das scheint eine Lösung zu sein:
    CSS:

    .shariff_closeX {
        position: absolute;
        right: 0;
        top: -25px;
        padding: 2px;
        text-decoration: none;
        box-shadow: none !important;
        border: none !important;
        cursor: pointer;
    }

    PHP:

    						<fieldset>
    							<legend class="shariff_mailform_headline">' . $mf_headline[$lang] . '</legend>
    							<div class="shariff_mailform_headline"><a href="' . get_permalink() . '" class="shariff_closeX"><svg xmlns="http://www.w3.org/2000/svg" width="21" height="20" viewBox="0 0 20 20"><path d="M10 0c-5.5 0-10 4.5-10 10s4.5 10 10 10 10-4.5 10-10-4.5-10-10-10zM10 18.1c-4.5 0-8.1-3.6-8.1-8.1s3.6-8.1 8.1-8.1 8.1 3.6 8.1 8.1-3.6 8.1-8.1 8.1z"/><path d="M13.1 5l-3.1 3.1-3.1-3.1-1.9 1.9 3.1 3.1-3.1 3.1 1.9 1.9 3.1-3.1 3.1 3.1 1.9-1.9-3.1-3.1 3.1-3.1z"/></svg></a></div>' . $mf_headinfo[$lang] . '
    							<input type="hidden" name="act" value="sendMail">
    							<input type="hidden" name="lang" value="' . $lang . '">
    							<p><label for="mailto">' . $mf_rcpt[$lang] . '</label><br>
    							<input type="text" name="mailto" id="mailto" value="' . $mf_content_mailto . '" size="27" placeholder="' . $mf_rcpt_ph[$lang] . '" required>' . $mf_to_error_html . '</p>
    							<p><label for="from">' . $mf_from[$lang] . $mf_optional_text . '</label><br>
    							<input type="email" name="from" id="from" value="' . $mf_content_from . '" size="27" placeholder="' . $mf_from_ph[$lang] . '" ' . $mf_sender_required .'>' . $mf_from_error_html . '</p>
    							<p><label for="sender">' . $mf_name[$lang] . '</label><br>
    							<input type="text" name="sender" id="sender" value="' . $mf_content_sender . '" size="27" placeholder="' . $mf_name_ph[$lang] . '"></p>
    							<p><label for="mail_comment">' . $mf_comment[$lang] . '</label><br>
    							<textarea id="mail_comment" placeholder="' . $mf_comment[$lang] . '" name="mail_comment" rows="4">' . $mf_content_mail_comment . '</textarea></p>
    							<input type="url" name="url" id="shariff_mailform_url" value="" size="27" placeholder="">
    						</fieldset>
    
Viewing 10 replies - 1 through 10 (of 10 total)
  • The topic ‘W3C’ is closed to new replies.