Opened 6 years ago
Closed 6 years ago
#8028 closed defect (bug) (fixed)
Javacript updates the Favourite/UnFavorite button incorrectly on Nouveau
Reported by: | joncadams | Owned by: | boonebgorges |
---|---|---|---|
Milestone: | 5.0.0 | Priority: | normal |
Severity: | normal | Version: | 3.0.0 |
Component: | Templates | Keywords: | |
Cc: |
Description
I think I found a bug on Nouveau:
the javascript gives the button a title attribute it did not have initially after updating with ajax, and does not update the tooltip as it should. to keep things consistent I made the following changes:
In bp-nouveau\js\buddypress-activity.js on line 464:
$( this ).prop( 'title', response.data.content );
should be:
$( this ).attr( 'data-bp-tooltip', response.data.content );
In bp-nouveau\includes\activity\ajax.php on line 145:
$response = array( 'content' => __( 'Favorite', 'buddypress' ) );
should be:
$response = array( 'content' => __( 'Mark as Favorite', 'buddypress' ) );
And in bp-nouveau\includes\activity\template-tags.php on line 378:
'link_text' => __( 'Favorite', 'buddypress' ),
should be:
'link_text' => __( 'Mark as Favorite', 'buddypress' ),
Change History (2)
Note: See
TracTickets for help on using
tickets.
Good catch, and good fix! Thank you.