Changeset 11606 for trunk/src/bp-core/admin/bp-core-admin-settings.php

Timestamp:

06/22/2017 08:09:27 PM (6 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-core/admin/bp-core-admin-settings.php (modified) (1 diff)

trunk/src/bp-core/admin/bp-core-admin-settings.php

@@ -272 +272 @@
     <?php if ( false === $file_exists ) : ?>
 
-        <a class="button" href="<?php bp_admin_url( 'admin.php?page=bb-forums-setup&repair=1' ); ?>" title="<?php esc_attr_e( 'Attempt to save a new config file.', 'buddypress' ); ?>"><?php _e( 'Repair', 'buddypress' ) ?></a>
+        <a class="button" href="<?php bp_admin_url( 'admin.php?page=bb-forums-setup&repair=1' ); ?>"><?php _e( 'Repair', 'buddypress' ) ?></a>
         <span class="attention"><?php _e( 'File does not exist', 'buddypress' ); ?></span>