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 158407 - Path cursor changes hotspot when cursor modifier changes
Path cursor changes hotspot when cursor modifier changes
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: User Interface
2.1.x
Other All
: High minor
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2004-11-15 23:11 UTC by khiraly
Modified: 2006-06-01 20:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
The proposed cursor (333 bytes, image/png)
2005-05-16 16:39 UTC, Michael Schumacher
Details
An illustration of the problem (1.72 KB, image/png)
2005-05-16 16:40 UTC, Michael Schumacher
Details
The illustration, animated (1.39 KB, image/gif)
2005-05-16 16:42 UTC, Michael Schumacher
Details

Description khiraly 2004-11-15 23:11:36 UTC
Hi!
There are 6 different tool types in the toolbox. 
The icon shows the actual used function, an arrow in the upper left side, the
actual tool in the bottom-right, and an an another in the upper-right.
If you want to edit on your Bezier-line, the icon changes one to another.
But, if you just nearly miss your Bezier-breakpoint, (which you wanted to edit),
the cursor will change to it's 'Do-Not-edit' shape, which differs from the other
cursor icons.
This bugs me a lot, because this 'do-not-edit' icon does not shape like the
normal one.
The center of the current 'do-not-edit' icon differs from the other icons
So I have made a mockup, how this 'do-not-edit' should look like
Because the current 'do-not-edit' cursor makes the editing really hard
http://www.epointsystem.org/~khiraly/gimp/pathCursorProposal.png
This one keeps to the original form, but gives also the state of your current action
and it centers like the others one (so the upper-left of the pointer).

Maybe this image explain a little bit more what Im trying to explain:
http://www.epointsystem.org/~khiraly/downtMatch.png

Where can I found the cursors image? (to modify it)
What are the others viewpoint? Is it a bug or not?

Best regards, 
 Khiraly
Comment 1 weskaggs 2005-02-25 18:51:38 UTC
The links given are no longer valid.  Please attach the images instead of
linking them if you want this bug report to remain under consideration.  (The
path tool ui probably could use an upgrade at some point, although nobody seems
inclined to work on it at the moment.)
Comment 2 Sven Neumann 2005-04-05 10:58:28 UTC
It would really help to have the images available. Khiraly, would you please
attach them to this bug-report? Thank you.
Comment 3 Michael Schumacher 2005-05-11 10:33:37 UTC
Khiraly, will we ever get access to these images again?
Comment 4 khiraly 2005-05-12 15:46:54 UTC
Sorry for removing the link from the website. Now (some month later), I
realised, that the downtMatch.png image are not self explanatory enough.
So I have made quickly a simple animation, what available here:
http://www.epointsystem.org/~khiraly/gimp/downtMatch.gif

Best regards, 
 Khiraly
Comment 5 Michael Schumacher 2005-05-16 16:39:35 UTC
Created attachment 46501 [details]
The proposed cursor
Comment 6 Michael Schumacher 2005-05-16 16:40:54 UTC
Created attachment 46502 [details]
An illustration of the problem
Comment 7 Michael Schumacher 2005-05-16 16:42:18 UTC
Created attachment 46503 [details]
The illustration, animated
Comment 8 Sven Neumann 2005-10-20 12:11:29 UTC
I agree that this is a bug. The hotspot of the cursor must not change when the
cursor modifier changes.
Comment 9 Michael Natterer 2006-06-01 20:31:46 UTC
Fixed in CVS:

2006-06-01  Michael Natterer  <mitch@gimp.org>

	* cursors/Makefile.am
	* cursors/modifier-bad.png
	* cursors/xbm/modifier-bad-mask.xbm
	* cursors/xbm/modifier-bad.xbm: new "bad" cursor
	modifier. Replaces the "bad" cursor.

	* cursors/gimp-tool-cursors.xcf: added it here too.

	* app/widgets/widgets-enums.h: added GIMP_CURSOR_MODIFIER_BAD.

	* app/widgets/gimpcursor.c: add the bad modifier. Leave the bad
	cursor there for now.

	* app/display/gimpdisplayshell-callbacks.c
	* app/tools/gimpaligntool.c
	* app/tools/gimpblendtool.c
	* app/tools/gimpbycolorselecttool.c
	* app/tools/gimpclonetool.c
	* app/tools/gimpcolortool.c
	* app/tools/gimpfliptool.c
	* app/tools/gimpiscissorstool.c
	* app/tools/gimpmovetool.c
	* app/tools/gimptransformtool.c
	* app/tools/gimpvectortool.c: use the modifier instead of the
	cursor. Fixes hotspot jumping when switching between normal and
	bad cursors. The changed cursor_update() functions even make more
	sense IMHO. Fixes bug #158407.