'Restore Brush' function on Brush Grab and Lasso buttons differs

[finticemo]

.. This is directly contrary to the help, which explicitly identifies the Restore Brush function as 'same as above'

Specifically, this is what occurs (I am assuming you have already grabbed a brush sometime in this GrafX2 session):

• Right Clicking on the Brush Grab button restores your brush. The Brush button shows the 'multicolor brush' symbol.

• Right Clicking on the Lasso button sets your brush to a 1px brush. However, the Brush button shows the 'multicolor brush' symbol, which is incorrect: this 1px brush behaves in all ways identical to the standard 1px brush, including adapting its color to the current Fore-color.

I am not sure whether it is the documentation that is wrong, or the code. But the documentation and the code clearly do not match up.

[Thomas Bernard]

@Pulkomandy : What should be done in your opinion ?

[PulkoMandy]

It seems to be a bug indeed. The "restore" should do the same thing on the two buttons.

Going back to the 1px brush is already easily accessible using the DEL key.

[yrizoud@…]

The reason is that the visual de-activation of the Lasso button calls Button_Unselect_lasso(), which assumes that the button had been activated through left-click, saving the "original paintbrush" in variable Paintbrush_shape_before_lasso.
I can't see an evident fix at the moment.

[Jordan Christiansen]

I'm working on debugging this. One thing I've noticed is that if you have a brush that you lasso'ed and you use the lasso tool again, from that point on the lasso restore function seems to work correctly, no matter how many times you change brushes or use the lasso tool.

For anyone who needs this to work right now, this could be a temporary workaround.

I'll try to figure this out and submit a patch.

[PulkoMandy]

Great, thanks for working on it! Let us know if you need any help with this.

[Jordan Christiansen]

I submitted a patch for this: ​https://gitlab.com/GrafX2/grafX2/merge_requests/109

[Jordan Christiansen]

My patch was merged, so this can be close now.