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 439749 - [patch] add maximize horizontally and vertically to double click on title bar actions
[patch] add maximize horizontally and vertically to double click on title bar...
Status: RESOLVED DUPLICATE of bug 358674
Product: metacity
Classification: Other
Component: general
trunk
Other All
: Normal enhancement
: 2.21.3
Assigned To: Metacity maintainers list
Metacity maintainers list
Depends on:
Blocks:
 
 
Reported: 2007-05-19 18:57 UTC by Jason Ribeiro
Modified: 2008-03-01 14:42 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
add maximize horizontally and vertically to double click on title bar actions (4.13 KB, patch)
2007-05-19 18:58 UTC, Jason Ribeiro
none Details | Review
I noticed I had tabs instead of spaces in the gconf description. Changed tabs to spaces. (4.15 KB, patch)
2007-05-19 19:02 UTC, Jason Ribeiro
none Details | Review
forgot to save core.h before running diff (4.84 KB, patch)
2007-05-19 19:13 UTC, Jason Ribeiro
none Details | Review
add maximize horizontally and vertically to double click on title bar actions (against revision 3476) (5.17 KB, patch)
2007-12-15 14:05 UTC, Jason Ribeiro
none Details | Review
Updated Patch (for v2.23.1/rev3616) (3.93 KB, patch)
2008-03-01 10:31 UTC, Christopher Roy Bratusek
none Details | Review
Fixed patch for common.h (3.89 KB, patch)
2008-03-01 10:35 UTC, Christopher Roy Bratusek
none Details | Review

Description Jason Ribeiro 2007-05-19 18:57:27 UTC
Keyboard shortcuts exist for maximizing a window vertically and horizontally.  This patch provides 2 new values for the /apps/metacity/general/action_double_click_titlebar gconf key: "toggle_maximize_horiz" and "toggle_maximize_vert".  The "toggle_maximize_vert" value causes the window to be maximized vertically when the title bar is double clicked.  Similarly, the "toggle_maximize_horiz" value causes the window to be maximized horizontally when the title bar is double clicked.

I have not patched control-center, but that should be easy as well if this is accepted.

This is my first patch, so let me know if I did anything wrong...
Comment 1 Jason Ribeiro 2007-05-19 18:58:20 UTC
Created attachment 88449 [details] [review]
add maximize horizontally and vertically to double click on title bar actions
Comment 2 Jason Ribeiro 2007-05-19 19:02:54 UTC
Created attachment 88450 [details] [review]
I noticed I had tabs instead of spaces in the gconf description.  Changed tabs to spaces.
Comment 3 Jason Ribeiro 2007-05-19 19:13:41 UTC
Created attachment 88451 [details] [review]
forgot to save core.h before running diff

ok, maybe I should sit on patches a little longer from now on before submitting them :)
Comment 4 Jason Ribeiro 2007-08-14 18:07:29 UTC
Just a reminder to have a look at this patch.  Thanks
Comment 5 Thomas Thurman 2007-12-10 21:35:11 UTC
Jason: Sorry to be so long getting back to you. The patch looks good (but I haven't run it yet), but I'm interested to know why you think it would be useful to be able to do this. Do you have a need to?
Comment 6 Thomas Thurman 2007-12-13 11:57:56 UTC
This person says he'd like vertical maximise on double-click titlebar, fwiw:
http://blogs.gnome.org/metacity/2007/12/12/2007-12-11-just-a-quick-roundup/#comments
Comment 7 Markus Berg 2007-12-15 12:27:21 UTC
I miss that feature.  I became addicted to it in the nineties on pre-CDE versions of Solaris.  However, I fully understand the reluctance to add more settings to Metacity.

With increasingly large screens, the correct solution might be more dialog between Metacity and the application, and to redefine "Maximize" to take the natural size of the document/window contents into consideration.  For example, clicking maximize on an Evince window would cause Metacity to ask Evince how large a window it would need to display the current document (given screen size constraints), and Metacity would then resize the Evince window to that size.  I believe the Macintosh used to have a system like this (maybe it still does?  I haven't used a Macintosh in more than ten years).
Comment 8 Jason Ribeiro 2007-12-15 13:45:28 UTC
My main use for the feature is when reading a document with illustrations or images.  I do not want to maximize in both directions because reading long lines is tiresome but I want to expand vertically so that I can view a full illustration without scrolling.  I never use the horizontal maximization, but added that option as well since it already existed for the keyboard shortcuts.

The patch is stale now, I'll upload an updated version in a bit.
Comment 9 Jason Ribeiro 2007-12-15 14:05:00 UTC
Created attachment 101007 [details] [review]
add maximize horizontally and vertically to double click on title bar actions (against revision 3476)
Comment 10 Thomas Thurman 2007-12-15 17:01:15 UTC
Jason: Hey, thanks for updating the patch!

Markus: There is actually a way for windows to hint to the WM they don't want to be wider than a certain width. I have an idea that the reason this doesn't actually happen in practice is that programs just don't do it, rather than that we don't support it (since I'm sure we do, and I've worked with the code that supports it). I will ask the evince people if you like and find out.

I imagine that even though you can do width constraint with full maximisation, it won't obviate your desire for vertical maximisation because a lot of programs don't set it.
Comment 11 Christopher Roy Bratusek 2008-03-01 10:31:00 UTC
Created attachment 106322 [details] [review]
Updated Patch (for v2.23.1/rev3616)

I like the Idea. I can't develop in C, so a review is recommended. I took the old patch and just updated it, so everything should be ok.
Comment 12 Christopher Roy Bratusek 2008-03-01 10:35:48 UTC
Created attachment 106323 [details] [review]
Fixed patch for common.h

there has a line beeing added to common.h, that shouldn't be added.
Comment 13 Thomas Thurman 2008-03-01 14:42:07 UTC
This turns out to be a duplicate... moving discussion over there.

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