Replace bookmark icon with something that highlights when selected

Bug #365923 reported by Ketil Wendelbo Aanensen
4
Affects Status Importance Assigned to Milestone
[Legacy] GNOME Activity Journal
Fix Released
High
Seif Lotfy

Bug Description

We need a GenericCellRenderer that works as a togglebutton with a star instead of a checkbox. The bookmark icon should be grey by default, then in colour when bookmark mode is active.

Revision history for this message
Seif Lotfy (seif) wrote :

Why not use a star?

Changed in gnome-zeitgeist:
status: New → Confirmed
importance: Undecided → High
Seif Lotfy (seif)
description: updated
Revision history for this message
Milan Bouchet-Valat (nalimilan) wrote :

Attached is a rough implementation of that cell renderer. It's based on GtkCellRendererToggle, except that it renders the cell using a GtkCellRendererPixbuf, merely transferring function calls, and setting the star icon as appropriate. Two star icons are taken from Firefox (what license exactly?).

Untested, but simple enough to be fixed in situation. I guess it should be integrated into the rest of the project to check that it works, else testing would be a useless pain.

Seif Lotfy (seif)
Changed in gnome-zeitgeist:
assignee: nobody → Siegfried Gevatter (RainCT) (rainct)
Seif Lotfy (seif)
Changed in gnome-zeitgeist:
assignee: Siegfried Gevatter (rainct) → Seif Lotfy (seif)
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.