When I go to a video in Safari (Mac) the message that comes has a checkbox (which I asume sets a cookie) and a <span> next to with the the text (mwe-embedplayer-do_not_warn_again) "Dismiss for now."
This should be changed into a <label> instead with the "for" attribute set to the name/id of the checkbox so that clicking the label will toggle the checkbox aswell (pointing the checkbox itself can be tricky at times..(usability)).
Krinkle
Version: unspecified
Severity: enhancement
I'm not sure if what's currently live on commons (loaded from prototype.wikimedia.org) is this - or that it is from the extension, but I found it over here:
The
779 $targetWarning.append(
780 $j('<span />')
781 .text( gM( 'mwe-embedplayer-do_not_warn_again' ) )
782 );
I guess should be changed to:
779 $targetWarning.append(
780 $j('<label />')
781 .text( gM( 'mwe-embedplayer-do_not_warn_again' ) )
782 .attr( 'for', 'ffwarn_' + embedPlayer.id )
783 );