Opened 8 years ago
Closed 5 years ago
#16 closed defect (fixed)
'Restore Brush' function on Brush Grab and Lasso buttons differs
|Reported by:||finticemo||Owned by:||PulkoMandy|
.. 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.
Change History (10)
comment:1 by , 5 years ago
comment:2 by , 5 years ago
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.
comment:3 by , 5 years ago
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.
comment:4 by , 5 years ago
|Milestone:||2.5 → 2.6|
comment:5 by , 5 years ago
|Status:||new → assigned|
comment:6 by , 5 years ago
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.
comment:7 by , 5 years ago
Great, thanks for working on it! Let us know if you need any help with this.
comment:8 by , 5 years ago
I submitted a patch for this: https://gitlab.com/GrafX2/grafX2/merge_requests/109
comment:9 by , 5 years ago
My patch was merged, so this can be close now.
comment:10 by , 5 years ago
|Status:||assigned → closed|
@Pulkomandy : What should be done in your opinion ?