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 352084 - Fullscreen menu item changes when selected
Fullscreen menu item changes when selected
Status: RESOLVED OBSOLETE
Product: gnome-games-superseded
Classification: Deprecated
Component: general
unspecified
Other Linux
: Low enhancement
: gnome-2-22
Assigned To: gnome-games-general-maint
GNOME Games maintainers
Depends on:
Blocks:
 
 
Reported: 2006-08-19 22:38 UTC by Mariano Suárez-Alvarez
Modified: 2012-11-14 01:28 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Make the Fullscreen menu item follow the HIG (10.85 KB, patch)
2006-08-20 00:18 UTC, Mariano Suárez-Alvarez
needs-work Details | Review

Description Mariano Suárez-Alvarez 2006-08-19 22:38:13 UTC
The Fullscreen menu item is changed into Leave Fullscreen when selected. The HIG (implicitely) suggests that Fullscreen be made a CheckMenuItem, though, and notes: "Never change the label of a check box menu item in response to the user selecting the item."

On the same line: the HIG says: "When your application is running in full screen mode, hide the menubar by default. [...] A Leave Fullscreen button should be placed in the upper right hand corner of the window. etc" (See <http://developer.gnome.org/projects/gup/hig/2.0/menus.html>; wouldn't it be nice if the HIG as posted had anchors to each point being made, for easy linking into it?)
Comment 1 Mariano Suárez-Alvarez 2006-08-20 00:16:09 UTC
Here comes a patch which does most of what the HIG wants, except for making the button disappear after 5 seconds of not moving the mouse or (more appropriately, since this app is one where the mouse is frequently used...) making it slide out of the window. 
Comment 2 Mariano Suárez-Alvarez 2006-08-20 00:18:01 UTC
Created attachment 71223 [details] [review]
Make the Fullscreen menu item follow the HIG

This additionally adds a bit of magic to keep -W -Wall happy.
Comment 3 Andreas Røsdal 2006-08-20 07:57:15 UTC
Hello Mariano,

Let me first start by saying that usability is important to gnome-games, and that you come with some good suggestions here.

However, since gnome-games is in UI and string freeze now, this change will have to wait until the 2.17.x development cycle, which begins when GNOME 2.16 has been released.

Further, we will need to discuss the proper implementation of the fullscreen mode. Your patch only changes same-gnome only, but the fullscreen menu item needs to be consistent in all the games. If this menu item behaves differently in different same-gnome, it will confuse users. This makes your patch uncommittable as it is now. 


Comment 4 Alan Horkan 2006-09-03 16:03:25 UTC
The label changing is mildly confusing for users but the more important underlying issue and the reason for that guidline is accessibility.  Screen readers have no way of knowing another label will appear later requiring them to go back and read it again.  

Comment 5 Mariano Suárez-Alvarez 2006-10-10 01:37:33 UTC
This is not NOTABUG. In any case, this should be changed into a global bug covering all games that offer fullscreening. 

"We will need to discuss the proper implementation [...]" in no way means the issue went away.

Reopening.
Comment 6 Christian Persch 2008-03-25 12:08:31 UTC
Re-assigning to new default owner.
Comment 7 Robert Ancell 2012-11-14 01:28:25 UTC
The games no-longer share this code. Please open against each game if still applicable.