Move the dropper from the toolbox to the color picker.

Bug #172236 reported by Daghead
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Wishlist
John Smith
Nominated for 0.46.x by Davjdag
Nominated for 0.47.x by Davjdag
Nominated for Old by Davjdag

Bug Description

The dropper really shouldn't be in the toolbox, and should be the last
color model tab in the color picker.
This would make it easier to pick color for specific elements in an object,
in addition to its just making more sense.

Viva Inkscape!

Tags: ui
nightrow (jb-benoit)
Changed in inkscape:
importance: Undecided → Wishlist
Bryce Harrington (bryce)
Changed in inkscape:
status: New → Confirmed
Revision history for this message
John Smith (john-smithi) wrote :

The dropper is probably a valid tool in the toolbox.
But a shortcut to it in the Fill/Stroke dialog color picker is also useful.
Have added a button in the color picker.
r11635.

Changed in inkscape:
assignee: nobody → John Smith (john-smithi)
status: Confirmed → Fix Committed
su_v (suv-lp)
Changed in inkscape:
milestone: none → 0.49
Revision history for this message
John Smith (john-smithi) wrote :

Commit r11652 - F/S color dropper is now a "one time" pick.
Once a color is picked, the dropper reverts to the previous tool.

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

A patch to ensure the Stroke page picker always applies the color to the objects Stroke (with any modifier key).
Fill page picker still works in the current way (<Shift> key changes from fill to stroke)

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

v2 patch
* Sifferent pPixmaps for dropper fill and stroke help clarify what is being set
* Both fill and stroke "one time" picker in F/S dialog don't ever allow setting "the other color" (via <Shift>key)

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

Patch tested ok on OS X 10.7.4 with r11888 (GTK+ 2.24.13, X11 and Quartz backend) - thank you both (John, Vlada) for this improvement!

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

Patch also tested ok with experimental GTK3 (GTK+/Quartz 3.6.2).

Revision history for this message
Vladimir Savic (vladimir-firefly-savic) wrote :

Any chance we'll see this committed soon? I think it's nice improvement...

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

Committed to trunk as r11894.

Changed in inkscape:
status: In Progress → Fix Committed
Revision history for this message
su_v (suv-lp) wrote :

> But a shortcut to it in the Fill/Stroke dialog color picker is also useful.
> Have added a button in the color picker.

Related follow-up report:
Bug #1086217 “Color picker button (color selector widget) needs to be context-aware“
<https://bugs.launchpad.net/inkscape/+bug/1086217>

Bryce Harrington (bryce)
Changed in inkscape:
status: Fix Committed → Fix Released
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.