GNOME Bugzilla – Bug 337376
Mouse cursor for moving windows is not ideal
Last modified: 2006-05-22 16:15:03 UTC
That bug has been opened on https://launchpad.net/distros/ubuntu/+source/metacity/+bug/38110 "The cursor for moving a panel applet in GNOME is the small hand, but this cursor is not used for moving a (metacity) window. Instead, a black + is used. Ideally, this should be changed for better consistency and looks."
Created attachment 63962 [details] [review] Makes the move cursor a hand instead of a cross The above patch changes the move cursor from a cross to hand. I think it looks better and it is also better for consistency reasons. OTOH, it is not so nice that the cursor changes when you maximize by double-clicking the titlebar.
Not sure I'd include the part of this patch changing the MetaCursor enum, those are "semantic names" right, so you could make two of them map to the same cursor but still keep them separate in case sometime we wanted to make them map to different. As to whether to change the cursor at all, consistency with the panel sounds fine. On Windows the cursor doesn't change at all for move operations, so that's another option.
(In reply to comment #2) > Not sure I'd include the part of this patch changing the MetaCursor enum, those > are "semantic names" right, so you could make two of them map to the same > cursor but still keep them separate in case sometime we wanted to make them map > to different. I don't know if I understand but I think that would be pretty confusing. You'd have code that told you one story "different MetaCursor when move or resize grabop" vs. observed behaviour: "same cursor for move and resize actions." You don't want a MetaCursor for each end every MetaGrabOp, do you? > As to whether to change the cursor at all, consistency with the panel sounds > fine. On Windows the cursor doesn't change at all for move operations, so > that's another option. Yeah, but then gnome-panel should change too, IMHO.
Hmmm.... I guess the debate all hinges on how you view the MetaCursor enum -- is it a listing of the different cursor states, or of different operations that may need a different cursor than normal? In the former, Björn's patch would be right, while if it is really the latter as Havoc alludes to, then we should probably add all the keyboard resizing operations (in case we ever want to make the cursor different for those) as Björn points out. Since it looks like the code was closer to the former already, let's stick with that.
Ok. Patch comitted. Please revert it if that was wrong. Not changing the cursor or keeping it as a plus are still options I suppose.