• 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

    Thread Starter Harry Milatz

    (@harry-milatz)

    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!

    Thread Starter Harry Milatz

    (@harry-milatz)

    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 https://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

    Thread Starter Harry Milatz

    (@harry-milatz)

    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

    Thread Starter Harry Milatz

    (@harry-milatz)

    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:)

    Thread Starter Harry Milatz

    (@harry-milatz)

    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="https://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.