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 150608 - add colors after palette editor cursor
add colors after palette editor cursor
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: User Interface
git master
Other All
: Normal enhancement
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2004-08-20 03:25 UTC by david gowers
Modified: 2005-07-13 16:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
changes the palette editor to add colors after the cursor (5.32 KB, patch)
2004-08-20 03:28 UTC, david gowers
needs-work Details | Review

Description david gowers 2004-08-20 03:25:58 UTC
this patch makes rearrangement of the palette relatively painless.
it does this by inserting added colors after the current cursor position rather
than the end of the list.

i'd also like to make it insert dropped colors after the color they were dropped
on, but don't know if i can obtain the coordinates of the drop in order to
determine this.
Comment 1 david gowers 2004-08-20 03:28:27 UTC
Created attachment 30773 [details] [review]
changes the palette editor to add colors after the cursor
Comment 2 Michael Natterer 2004-12-31 17:30:26 UTC
Thanks to fixing bug #150030, gimp_palette_add_entry() has a "position"
parameter now and dropped colors are inserted at the right place.

It seems like the logical next step to apply the rest of this
patch, however I'm unsure if the color should be inserted
*at* or *after* the currently selected color.

Opinions?
Comment 3 Sven Neumann 2004-12-31 17:39:08 UTC
IMO the drop cursor should indicate where the color is inserted, that is between
two colors.
Comment 4 david gowers 2005-01-01 00:25:26 UTC
I wanted to implement that but didn't understand how. Probably best to insert
before the color it was dropped on if it was dropped on the left half of a color
panel, and after if dropped on the right half.
Comment 5 Michael Natterer 2005-01-02 18:24:47 UTC
Note we don't talk about DND here. That's already implemented in HEAD
and just needs to be refined. This bug is about inserting new colors
before or after the cursor position.
Comment 6 david gowers 2005-01-04 00:32:10 UTC
I think after more common (most obviously for adding to the end of the palette.). 
Comment 7 Michael Natterer 2005-07-13 16:24:11 UTC
Fixed in CVS:

2005-07-13  Michael Natterer  <mitch@gimp.org>

	* app/widgets/gimppaletteeditor.c (gimp_palette_editor_pick_color):
	add colors after the cursor. Fixes bug #150608.