GNOME Bugzilla – Bug 664956
Last modified: 2011-12-26 03:07:15 UTC
Remove statusbar. Move any remaining content (e.g. the clock) into the toolbar or into the game view.
Working on it...
Created attachment 204198 [details] [review]
Removes statusbar and moves content to toolbar
Review of attachment 204198 [details] [review]:
Patch is generally good. Some minor things that should be done when doing patches in the future:
- Commit comment should match style of existing commits, so in this case it should have had a "mahjongg: " prefix
- Coding style should be consistent and match existing style. In this case there were some spaces missing before the '(' in the function calls.
I've pushed this patch to master with some other changes that weren't specified in the bug report. Thanks for your work!