Allow dropper tool to edit current custom swatch fill

Bug #1049481 reported by su_v
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
In Progress
Wishlist
John Smith

Bug Description

Follow-up to:
Bug #239392 “Missing Color Picker in Fill and Stroke Dialog”
Related to:
Bug #171170 “Use color picker to add colors to palette”

When selecting an object with a custom swatch, the Fill&Stroke dialog in trunk shows the list of available swatches, and the color selector widget below to edit the color of the current swatch.

Expected behavior:
Clicking on the dropper tool icon in the color selector widget of the current swatch fill will apply the picked color to the current swatch fill (as if the color sliders had been adjusted).

Actual behavior (r11664):
Instead of modifying the color of the current swatch fill, the picked color replaces the swatch with a flat color.

Proposal:
When calling the color picker from the color selector widget of a swatch fill, allow it to edit the color of the current swatch by picking a color from the canvas (instead of switching from swatch color to flat color).

Alternative (somewhat better then current situation):
Hide the dropper tool icon in the special variant of the fill&stroke dialog for swatch fills.

Tags: color ui dropper
Revision history for this message
su_v (suv-lp) wrote :
John Smith (john-smithi)
tags: added: dropper
jazzynico (jazzynico)
Changed in inkscape:
status: New → Confirmed
Revision history for this message
John Smith (john-smithi) wrote :

Attached patch should work as per proposal above, please test.

Revision history for this message
su_v (suv-lp) wrote :

Patch looks good to me in initial tests (OS X 10.7.4, GTK+/X11 2.24.13, r11806) - would be great though if it gets tested by other (experienced) users who work with trunk builds on complex drawings, and make use of the 'Auto' palette feature.

Revision history for this message
su_v (suv-lp) wrote :

Likely this will only get tested by other users once it's committed in current trunk.

Revision history for this message
insaner (insaner) wrote :

not sure whether this goes here, in #239392 or #172236

but in r11729, i cant select the stroke color with the color picker, having the stroke color selection editor and clicking on the color picker makes changes to the fill, not the stroke.

Revision history for this message
su_v (suv-lp) wrote :

insaner wrote
> not sure whether this goes here, in #239392 or #172236
>
> but in r11729, i cant select the stroke color with the color picker (…)

Unless you are testing the patch from comment #2, it doesn't go here (the proposed patch for the requested feature here in bug #1049481 has not been committed yet).

Note that this report is about changing the color of custom swatches (i.e. named paint servers in the 'Auto' palette) with the dropper tool, not about normal fill or stroke colors.

Not to have to use 'Shift' to affect the stroke color with the dropper tool when called as "one time" pick from Fill&Stroke would be related to bug #172236 (either comment there or file a follow-up report requested this feature). It's troubling though, because either solution is inconsistent in some way (IMHO).

Revision history for this message
su_v (suv-lp) wrote :

insaner wrote:
> not sure whether this goes here, in #239392 or #172236
>
> but in r11729, i cant select the stroke color with the color picker (…)

Ah, now I realize that the dropper support for custom swatch colors for the 'Stroke paint' is also missing in the patch from comment #2.

@John - based on how the patch works when using the 'one time' dropper tool from the 'Swatch fill' variant of the 'Fill' pane, could you also implement it on the 'Swatch Fill' variant of the 'Stroke paint' pane, so that the 'one time' dropper tool there modifies the custom swatch color in use for the stroke of the current selection?

Revision history for this message
su_v (suv-lp) wrote :

@John - what do you think about the issue raised by insaner (comment #5) [1], and about the patch you wrote earlier for better swatch support?

[1] could possibly also be the cause for <http://article.gmane.org/gmane.comp.graphics.inkscape.devel/39418> though the user doesn't disclose whether he actually has a development snapshot build installed.

Revision history for this message
John Smith (john-smithi) wrote :

A fix for the problem insaner mentions (comment #5) has been committed (see bug #172236).
This patch should work together with that commit.

One concern, what is the preferred behavior with the dropper when a non-solid (gradient) swatch is selected ?

Changed in inkscape:
assignee: nobody → John Smith (john-smithi)
status: Confirmed → In Progress
Revision history for this message
su_v (suv-lp) wrote :

> what is the preferred behavior with the dropper
> when a non-solid (gradient) swatch is selected ?

Initially I thought this could be addressed by limiting this new feature to the scope of the 'one-time' dropper tool (i.e. when called from the button in the color selector widget in the Fill&Stroke dialog): thus it would either modify the color of the solid swatch, or the stop color of the selected gradient stop (be it a regular gradient or a gradient swatch). The normal mode of the dropper tool would still replace the swatch with a regular flat color (like it does with objects which use a gradient paint server).

Unfortunately the content of the Fill&Stroke panel for gradient swatches is soemwhat inconsistent in this regard (it displays the color selector even though only the first stop of the gradient swatch is actually modifiable), and thus currently doesn't allow above solution.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.