GNOME Bugzilla – Bug 91338
[PATCH] use wheel scroll in title bar
Last modified: 2005-12-31 02:35:37 UTC
this patch allows you to use wheel scroll on title-bar: scrollup - shade window scrolldown - unshade window alt+scrollup - move window to next workspace alt+scrolldown - move window to pre workspace
Created attachment 10622 [details] [review] patch that allows you use to wheel scroll
Created attachment 10623 [details] [review] Same as gianluigi's patch but for the metacity 2.4.0 tarball
I wonder if this would result in confusion if you were scrolling your web page or something and moved the mouse onto the titlebar?
Created attachment 10641 [details] [review] Added a GConf key (default to false), this patch also contains gianluigi's patch
Hmm... I doubt it would be that easy to accidentally 'scroll' the title bar in the act of trying to scroll the main window contents, so it might be worth trying it out if people think it's a valuable feature. (It doesn't do much for me, though, I have to confess, I find it far easier to double-click.)
I agree with Calum, anyway I have done a patch that adds a GConf key to enable/disable the wheel scroll. IMHO gianluigi's patch is great, it is so comfortable! So, I pray havoc to apply the latest cumulative patch I made, to metacity HEAD ;)
Well, un/shading doesn't give me much, as doubleclick is just as handy. Scrolling workspaces with ALT doesn't give me much either, as I can already conveniently switch workspaces when using the keyboard (grabbing the titlebar and pressing the workspace-switch shortcut). So to me the most practically default function would be to switch workspaces so I could move windows to another workspace without using the keyboard, rightclick menu or the tiny workspace switcher. If this would be too dangerous(?) for beginners, maybe "scrollwheel behaviour on titlebar" would make a good preference just like doubleclick behaviour? So people could for example choose combinations like doubleclick to maximize and scrollwheel to shade.
Ok to quote the metacity readme file: ______________________________________________________________ Q: Will you add my feature? A: If it makes sense to turn on unconditionally, or is genuinely a harmless preference that I would not be embarrassed to put in a simple, uncluttered, user-friendly configuration dialog. .... Just be prepared to hear the above objections if your patch adds some crack-ridden configuration option. ________________________________________________________________ My point is that this is not a feature that warrants a preference. Either it is the right behavior and it should be included, or its wrong and shouldn't be included. That said I haven't tried it and have no real opinion on the feature, other than it shouldn't be configurable.
bordoley: I made it configurable so Havoc has 2 option - a "plain" patch and a configurable patch. But hp doesn't say nothing. This makes me sad :(
Pier, I haven't had time to go through metacity bugs. I do it every couple weeks usually. I need to think about this. One thing I thought about recently is that mozilla uses Alt+scrollwheel to move back/forward pages, and that has always bothered me (since I'll hit alt for some other reason, while I'm scrolling, and lose my page)
OK Havoc, I needed only an answer (Gianluigi is in vacation so I must reply to the post of this bugs), and now I have one :) Btw I don't known when you go through Metacity's bugs, so scuse me if I made something wrong. I saw Mozilla, it uses ALT+wheel but it doesn't matter IMHO because it's hard to scroll on the title bar instead of a Mozilla window. A user should have control on what he does. Scrolling is much comfortable than double clicking (IMHO).
So, I don't like this idea. People accidentally use the scroll wheel *all the time*. Someone unfamiliar with window shading would be left wondering what happened to their app. Also, I don't think its obviously intuitive.
Alex: Is this the reason I made it configurable. I think this feature is good, but making it configurable (default on false) this avoids these problems.
*** Bug 92617 has been marked as a duplicate of this bug. ***
Attack of the Bug Squad: 1. Sean Middleditch did a patch for this, id=10926. I'm adding this comment here because the bug he attached the patch to is a dup and I don't want the code to get lost. 2. Cc += alex@ximian.com. I think Alex accidentally removed Louie's "triaged" keyword, so I'm adding it again. Many apologies if that's wrong.
I think I have to go with Bordoley's quote from the README. I don't think it's suitable to enable this by default, and it's not really in the spirit of metacity to have this kind of optional feature. Standard disclaimer: http://pobox.com/~hp/features.html Thanks for taking the time to submit a patch, of course.
*** Bug 116024 has been marked as a duplicate of this bug. ***
*** Bug 173011 has been marked as a duplicate of this bug. ***
*** Bug 324513 has been marked as a duplicate of this bug. ***
*** Bug 325352 has been marked as a duplicate of this bug. ***