Changeset 11606 for trunk/src/bp-templates/bp-legacy/buddypress/members/single/messages/notices-loop.php

Timestamp:

06/22/2017 08:09:27 PM (8 years ago)

Author:

djpaul

Message:

HTML <a> elements should never have title attributes.

Accessible support is too inconsistent across software and platforms.
In a small number of places (mainly legacy templates) where the link text was a meaningless value, the attribute was swapped out for aria-label, which reliably overides the link text readout.

Props rianrietveld

File:

• trunk/src/bp-templates/bp-legacy/buddypress/members/single/messages/notices-loop.php (modified) (1 diff)

trunk/src/bp-templates/bp-legacy/buddypress/members/single/messages/notices-loop.php

@@ -75 +75 @@
                 <td width="10%">
                     <a class="button" href="<?php bp_message_activate_deactivate_link(); ?>" class="confirm"><?php bp_message_activate_deactivate_text(); ?></a>
-                    <a class="button" href="<?php bp_message_notice_delete_link(); ?>" class="confirm" title="<?php esc_attr_e( "Delete Message", 'buddypress' ); ?>">x</a>
+                    <a class="button" href="<?php bp_message_notice_delete_link(); ?>" class="confirm" aria-label="<?php esc_attr_e( "Delete Message", 'buddypress' ); ?>">x</a>
                 </td>
             </tr>