#3618 closed defect (bug) (fixed)
UI issue – ”Remove Favorite” title
| Reported by: | wedeem | Owned by: | |
|---|---|---|---|
| Priority: | trivial | Milestone: | 1.5.2 |
| Component: | Core | Version: | 1.5 |
| Severity: | trivial | Keywords: | ui-feedback |
| Cc: |
Description
After marking a item as a “Favorite”, when you hover over “Remove Favorite” the title is the same as “Favorite”. The title being “Mark as Favorite”.
The title should change once marked as favorite.
Change History (6)
#1
@
15 years ago
- Milestone Awaiting Review → 1.5.1
- Priority normal → trivial
- Severity normal → trivial
#3
@
15 years ago
The patch would need to add the localised strings to bp_dtheme_enqueue_scripts()
Strictly speaking, since the AJAX handler just returns the button text, you could use that same text to populate the title attribute. But that seems a bit hackish in a couple ways, so adding separate strings seems better (even if they say the same thing).
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
The javascript that we have for marking/unmarking favorites should be revisited on a broader scale at some point. But, for a bugfix release, we could consider adding a line around 187 in global.js that changes this title text. Patches welcome, otherwise this one's getting booted.