On Thu, Nov 17, 2016 at 11:07 AM, Tavmjong Bah <tavmjong@...8...> wrote:
Thanks for your feedback!
On Wed, 2016-11-16 at 09:55 +0100, Sebastian Zartner wrote:
It took a bit, though here's my feedback. Generally the mesh gradient tool already works very well. So, congratulations and thank you a lot for that!
- Hard to associate handles with nodes When bezier handles are near to each other it is hard to
distinguish which one belongs to which corner node. Therefore bezier handles and their related corner nodes should be connected through a line like it's done within the vector tool.
I am working on a solution where hovering over a corner node will temporarily highlight the associated handles. Drawing lines doesn't work well when the mesh is dense. Jabier has a branch that changes the color of the handles for selected corner nodes. If I recall correctly (I am having trouble compiling it now) it worked well for sparse meshes but not well for dense meshes.
Looking forward to that. You could try improving that, Jabier.
- Mesh tool is separate from other gradients The mesh gradient tool is just another type of gradient, so it
should be part of the existing gradient tool instead of having a separate button within the toolbar.
This will be hard to do. The editing the mesh gradient acts quite differently from the linear and radial gradients.
Jabier had made a bunch of improvements related to hiding widgets that are conceptually N/A. It works great here! Take a look at Spray tool mode change behavior (Change mode from Spray copies -> Spray objects into single path). It hides what is unrelated to currently selected mode. I'd move Create Mesh Gradient and Create Conical Gradient just behind Create Linear Gradient and Create Radial Gradient. Followed by what it affects (Fill or Stroke). Then change all other fields related to what type of gradient is selected in the first place.
- Linear and radial gradient tool allows to change mesh gradients but
not vice versa When the linear and radial gradient tool is selected, you can also change mesh gradients, but when the mesh gradient tool is selected, you cannot edit linear or radial gradients. This inconsistency should be removed, i.e. you should always be able to edit the gradients. This goes hand in hand with the above idea of integrating the mesh tool into the existing gradient tool. (This also applies to conical gradients.)
This may be possible but may require a lot of work.
I am for removing possibility to change gradients outside gradient tool. It is easy enough to learn shortcuts and we already have dedicated tool for that job.
Thanks! Glad you liked it.
Tav
Sebastian
This tool improves Inkscape's value as illustrating tool sooo much... Thanks, Tav!
Vlada