Please use the class "element-invisible" instead of hiding the label to improve the usability for screenreaders:

Change form_placeholder.js at line 57

             if (!$textfield.attr('placeholder')) {
             $textfield.attr('placeholder', $.trim($label.text()));
-            $label.hide();
+            $label.addClass('element-invisible');
           }
 
           // Fallback support for older browsers.

(Sorry – i'm not yet able to create a real working patch …)

CommentFileSizeAuthor
#5 use_class-2779605-5.patch420 bytesrafalenden

Comments

tfranz created an issue. See original summary.

  • rafal.enden committed 80f7c0a on 7.x-1.x
    Issue #2779605 by tfranz: Use class "element-invisible" instead of style...
rafalenden’s picture

Status: Active » Fixed

Thanks tfranz!

rafalenden’s picture

Version: 7.x-1.x-dev » 8.x-1.x-dev
Status: Fixed » Patch (to be ported)

We need to port it to D8 version.

rafalenden’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new420 bytes

  • rafal.enden committed c3a6bff on 8.x-1.x
    Issue #2779605 by rafal.enden, tfranz: Use class "element-invisible"...
rafalenden’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.