use the proper -panel icon for the default avatar

Bug #550956 reported by David Barth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Me Menu
Invalid
Medium
Omer Akram

Bug Description

Use the nicer panel specific icon to be better integrated with the Ubuntu theme.

David Barth (dbarth)
Changed in indicator-me:
assignee: nobody → David Barth (dbarth)
importance: Undecided → Medium
milestone: none → ubuntu-10.04-beta-2
status: New → Triaged
Revision history for this message
Ted Gould (ted) wrote : Re: [Bug 550956] [NEW] use image-helper (-panel) for the default avatar

Yes, but, don't use the helper here. It should only be used for icons
that are on the panel as it has other behaviors that are unique to
there. You should use a GIcon if you just want the fallbacks.

Revision history for this message
David Barth (dbarth) wrote : Re: use image-helper (-panel) for the default avatar

This patch will ensure gnome-about-me is consistent with our (upcoming) changes.

David Barth (dbarth)
summary: - use image-helper (-panel) for the default avatar
+ use the proper -panel icon for the default avatar
David Barth (dbarth)
Changed in indicator-me:
status: Triaged → In Progress
Revision history for this message
David Barth (dbarth) wrote :

Fix to use the proper icon in the me menu

Changed in indicator-me:
status: In Progress → Fix Committed
Revision history for this message
David Barth (dbarth) wrote :

A better version is now in trunk, that uses the stock_person fallback icon, if the -panel one is not present. See lp:indicator-me (revision 94)

David Barth (dbarth)
Changed in indicator-me:
milestone: ubuntu-10.04-beta-2 → ubuntu-10.04
Revision history for this message
Kenneth Wimer (kwwii) wrote :

@David Barth: please let me know when this goes into Lucid as I will need to test/check it.

Revision history for this message
David Barth (dbarth) wrote :

Trunk now contains just the patch to get that icon, and no other change for the frame decoration. Too late for Lucid if we still need to do a round trip at that stage. It's ready for Maverick.

Revision history for this message
Omer Akram (om26er) wrote :

closing as invalid since MeMenu no longer shows the user avatar

Changed in indicator-me:
assignee: David Barth (dbarth) → Omer Akram (om26er)
status: Fix Committed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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