Opened 8 years ago
Closed 8 years ago
#8028 closed defect (bug) (fixed)
Javacript updates the Favourite/UnFavorite button incorrectly on Nouveau
| Reported by: | joncadams | Owned by: | boonebgorges |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.0.0 |
| Component: | Templates | Version: | 3.0.0 |
| Severity: | normal | 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Good catch, and good fix! Thank you.