Spray tool pressure sensitivity broken?

Bug #1459845 reported by Lucian on 2015-05-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Medium
jazzynico

Bug Description

I just acquired a Wacom Bamboo CTL-471 to use with Inkscape. Thankfully I've had few problems getting it to work. The Calligraphy tool and the tweak tool work with pressure sensitivity as expect ( a joy to use!). But I've noticed that the Spray tool pays no attention to the "Use pressure of input device..." setting. The amount sprayed is determined by the value in the "Amount" box, and nothing else. Pressure on the pen makes no difference at all that I can see. Given that the difference is very clear with the other tools, I'm guessing there is a bug with the Spray tool implementation somehow.

I'm running Inkscape 0.91 on Mac OS 10.10.3.

su_v (suv-lp) on 2015-05-29
tags: added: spray tablet
su_v (suv-lp) wrote :

Reproduced with Inkscape 0.48.5, 0.91 r13725 and 0.91+devel r14200 on OS X 10.7.5, using a Wacom Bamboo CTH-470:
There is no noticeable effect of pen pressure on the amount of the sprayed objects (copy or clone mode does not make a difference).

Pressure sensitivity with this combination (Inkscape on OS X under X11) otherwise works ok (e.g. with calligraphy tool).

Changed in inkscape:
importance: Undecided → Medium
status: New → Confirmed
jazzynico (jazzynico) wrote :

The Spray tool share some code (including the pressure part) with the Tweak code. Could you please test the Tweak tool and confirm the pressure work or not?

jazzynico (jazzynico) wrote :

Also reproduced on Crunchbang Waldorf, Inkscape trunk rev.14204, with a Bamboo fun CTE 450 tablet.
Pressure does work as expected with the Calligraphy and Tweak tools, but not with the Spray.
I wonder if the pressure option could be linked to a parameter which is no longer used (the Spray tool was originally designed to support some additional features).
Investigations in progress.

Changed in inkscape:
status: Confirmed → Triaged
jazzynico (jazzynico) wrote :

> I wonder if the pressure option could be linked to a parameter which is no longer used

Confirmed. The pressure is linked to a Force parameter which is not used for spraying. I suggest we link it to the population parameter (Amount in the UI) instead.

jazzynico (jazzynico) wrote :

The attached patch links the tablet pressure to the Amount setting and removes some unused code in the spray tool.
Tested on Crunchbang Waldorf with Inkscape trunk rev. 14205.

Tests and comments are welcome!

Changed in inkscape:
assignee: nobody → jazzynico (jazzynico)
milestone: none → 0.92
status: Triaged → In Progress
jazzynico (jazzynico) wrote :

Patch committed trunk rev. 14210.

If nobody complains about the link between pressure and the Amount setting, the patch can also be backported to the 0.91.x branch.

Changed in inkscape:
status: In Progress → Fix Committed
tags: added: backport-proposed
su_v (suv-lp) wrote :

Fix backported to 0.91.x in rev 13800.

Changed in inkscape:
milestone: 0.92 → 0.91.1
tags: removed: backport-proposed
jazzynico (jazzynico) on 2017-01-22
Changed in inkscape:
milestone: 0.91.1 → 0.92
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers