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 664953 - Remove statusbar
Remove statusbar
Product: gnome-games-superseded
Classification: Deprecated
Component: gnomine
Other Linux
: Normal normal
: ---
Assigned To: GNOME Games maintainers
GNOME Games maintainers
Depends on:
Reported: 2011-11-27 22:49 UTC by Robert Ancell
Modified: 2012-01-30 22:43 UTC
See Also:
GNOME target: ---
GNOME version: ---

This patch will remove the status bar and the top and bottom separators (3.78 KB, patch)
2011-12-18 04:59 UTC, Sophia Yu
committed Details | Review
Screenshot of the new GUI (16.88 KB, image/png)
2011-12-18 05:01 UTC, Sophia Yu
Image showing the toolbar in gnotravex (36.16 KB, image/png)
2011-12-19 06:57 UTC, Robert Ancell
Add toolbar for Gnomine (8.69 KB, patch)
2012-01-20 10:09 UTC, Sophia Yu
needs-work Details | Review
Screenshot of updated Gnomine with the toolbar (26.72 KB, image/png)
2012-01-20 10:10 UTC, Sophia Yu
This is my second attempt to fix this bug. Also check the attached screenshot (11.49 KB, patch)
2012-01-30 09:07 UTC, Sophia Yu
committed Details | Review
New Gnomine (25.31 KB, image/png)
2012-01-30 09:09 UTC, Sophia Yu

Description Robert Ancell 2011-11-27 22:49:49 UTC
Remove statusbar.  Move any remaining content (e.g. the clock) into the toolbar or into the game view.
Comment 1 Sophia Yu 2011-12-11 08:46:05 UTC
I want to do this task.
Comment 2 Robert Ancell 2011-12-12 04:03:21 UTC
Thanks Sophia!  Please post status patches into this bug report.
Comment 3 Sophia Yu 2011-12-18 04:59:36 UTC
Created attachment 203768 [details] [review]
This patch will remove the status bar and the top and bottom separators

Please take a look at my patch. This is my first Vala-related patch. Basically I just merged the status bar with the New Game button. The flags of the mines is shown at the left of the button, the clock at the right. I also removed the code for creating the top and bottom separators because I thought it looked better than before. I hope I didn't violate any GUI design guide.
Comment 4 Sophia Yu 2011-12-18 05:01:02 UTC
Created attachment 203769 [details]
Screenshot of the new GUI

Please check the new GUI with the proposed patch. Thanks!
Comment 5 Robert Ancell 2011-12-19 03:21:08 UTC
Review of attachment 203768 [details] [review]:

The patch looks correct, and works well.  I agree removing the separators was a good idea.

One minor thing on the patch, in the future can you set the commit comment to "gnomine: ..." so it is clear this patch applies to gnomine and not the other games?
Comment 6 Robert Ancell 2011-12-19 03:42:39 UTC
Hi Sophia,

It looks much better already. I've applied the patch, congratulations!  There is one more piece of work that can be done if you're interested - to convert the elements at the top into a proper toolbar.  You can have a look at how this is done in gnotravex.  I'd recommend you make a new game button that has the face as the icon, add a pause button and put the time / flag count on the right.
Comment 7 Sophia Yu 2011-12-19 05:06:29 UTC
Hi Robert,

I don't understand what you meant by the toolbar. Did you mean something like what KMines looks Also, I didn't see a toolbar is gnotravex. I'm afraid I'm quite lost.
Comment 8 Robert Ancell 2011-12-19 06:57:01 UTC
Created attachment 203840 [details]
Image showing the toolbar in gnotravex

This is the toolbar in gnotravex.  Note that it's only in the master version.
Comment 9 Robert Ancell 2012-01-17 00:30:59 UTC
Hi Sophia,

Do you plan to do the toolbar change?  Any more information required?

Comment 10 Sophia Yu 2012-01-18 04:21:32 UTC
Yes. I want to give it a try today or tomorrow. I don't need any more info ATM.
Comment 11 Sophia Yu 2012-01-20 10:09:07 UTC
Created attachment 205684 [details] [review]
Add toolbar for Gnomine

I need feedback on how to deal with the face icon, such as where it should be positioned and do we really need it?
Comment 12 Sophia Yu 2012-01-20 10:10:09 UTC
Created attachment 205685 [details]
Screenshot of updated Gnomine with the toolbar
Comment 13 Robert Ancell 2012-01-21 07:05:20 UTC
Review of attachment 205684 [details] [review]:

What I'd recommend you do is use the face image for the new game button.  It's not strictly required that we keep it but it's a fun difference from the standard widgets and a nod to the past of the application.  Probably the easiest way to do this is to create a new Gtk.ToolButton and pack the image and "New Game" label into that.  Also you should mark the actions as important so the labels show in the toolbar.
Comment 14 Sophia Yu 2012-01-21 07:45:20 UTC
Thanks a lot for the suggestions! I will try to improve it.
Comment 15 Sophia Yu 2012-01-30 09:07:42 UTC
Created attachment 206399 [details] [review]
This is my second attempt to fix this bug. Also check the attached screenshot
Comment 16 Sophia Yu 2012-01-30 09:09:57 UTC
Created attachment 206401 [details]
New Gnomine
Comment 17 Robert Ancell 2012-01-30 22:43:21 UTC
Review of attachment 206399 [details] [review]:

Great work.  I've committed this with a few fixes:
- Marked the new game and hint buttons as important so they have labels
- Don't get the new game button using toolbar.get_nth_item (0) - get it by name
- Some spacing fixes (don't think they were caused by this patch though).