After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 348989 - Make use of view-fullscreen icon
Make use of view-fullscreen icon
Status: RESOLVED DUPLICATE of bug 357143
Product: totem
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2006-07-27 20:53 UTC by Michael Monreal
Modified: 2006-11-01 10:42 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Patch against HEAD (696 bytes, patch)
2006-07-27 20:55 UTC, Michael Monreal
needs-work Details | Review
Additional icons (8.12 KB, patch)
2006-07-27 21:33 UTC, Michael Monreal
rejected Details | Review
Patch against HEAD, 2nd Try (1.97 KB, patch)
2006-08-26 18:01 UTC, Michael Monreal
none Details | Review

Description Michael Monreal 2006-07-27 20:53:08 UTC
There is (at least now in 2.15.x) a view-fullscreen icon that would better fit the "Fullscreen" menu entry.
Comment 1 Michael Monreal 2006-07-27 20:55:29 UTC
Created attachment 69766 [details] [review]
Patch against HEAD

This is for use with g-i-t 2.15.x and uses the new icon naming scheme.
Comment 2 Michael Monreal 2006-07-27 21:33:22 UTC
Created attachment 69769 [details] [review]
Additional icons

This converts all icons (from totem.glade) that have a matching name in the icon spec to the new name. Works nice with g-i-t 2.15.x and tango. This now uses different icons for volume up and down.
Comment 3 Bastien Nocera 2006-07-28 11:18:24 UTC
This would mean stopping using the gtk+ stock icons for a lot of those, which I'm not happy with. Most of the icons should fallback back nicely already, and I'd like the "view-fullscreen" to use the framework in totem-stock-icons.c
Comment 4 Michael Monreal 2006-08-26 18:01:17 UTC
Created attachment 71666 [details] [review]
Patch against HEAD, 2nd Try

Well, yesterday I had to play a few movies and I remember filing this bug. Seeing that it is still open and the 2.16 release just around the corner I had a look myself.

I *think* this is what you want, but that produced horrible scaling, because the menu icons are set to use those icons in size of GTK_ICON_SIZE_BUTTON. This is bad so I changed to GTK_ICON_SIZE_MENU. It may be needed to case this for menu/button icons but at least here I can't see a difference on the exit fullscreen button, it uses the 16x16 icon in both cases. Take screenshot and fullscreen really look better this way though.
Comment 5 Bastien Nocera 2006-11-01 10:42:55 UTC
This was all fixed in #357143. Thanks for the patches though.

*** This bug has been marked as a duplicate of 357143 ***