Re: [patch] middle click panning and related
- From: Lars Clausen <lrclause cs uiuc edu>
- To: dia-list gnome org
- Subject: Re: [patch] middle click panning and related
- Date: Mon, 11 Aug 2003 15:12:03 -0500
On 15 Jul 2003, Krzysztof Foltman wrote:
Contains:
patch 1. middle button panning + SHIFT and CONTROL functions for
scroll wheel (SHIFT = x scroll, CONTROL = zoom) + shift click for
sticky tools (in fact, it reverses tool stickyness) - I had to replace
"toggled" signal (which isn't triggered on reclicking same button with
different SHIFT state) with "clicked"
I was about to apply this, but some of the details in it are still too
messy. For the panning, I still don't like having a new global variable
for the previous tool. It should instead be passed to create_scroll_tool
and stored in the ScrollTool structure. For the tool stickiness reversing
-- which UI-wise is nice -- adding an extra parameter to a bunch of central
calls just for that is ugly -- better to pass in at least the full set of
modifiers so they could be used for other purposes.
Sorry to delay so long, but it's harder to review a complex, multi-function
patch than a simple one, and so more easily gets pushed behind other
stuff.
Some notes for patch-creators: Please keep each patch to one purpose. If I
agreed with one part of this patch and not the other, I would have to put
in a lot of extra work to single those parts out. Also, please refrain
from doing cleanup together with functional patches, it makes reviewing
harder. Cleanup as a stand-alone patch woud be fine.
patch 2. some name changes for flowchart sheet,
http://bugzilla.gnome.org/show_bug.cgi?id=115238 (doesn't fix the bug
completely, just the naming part).
This one is fine and committed (though with a thought attached in the
ChangeLog).
-Lars
--
Lars Clausen (http://shasta.cs.uiuc.edu/~lrclause)| HÃrdgrim of Numenor
"I do not agree with a word that you say, but I |----------------------------
will defend to the death your right to say it." | Where are we going, and
--Evelyn Beatrice Hall paraphrasing Voltaire | what's with the handbasket?
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]