Change -h Tool Command Flag to Always Impact Secondary Brush#2560
Change -h Tool Command Flag to Always Impact Secondary Brush#2560Zeranny wants to merge 3 commits intoIntellectualSites:mainfrom
Conversation
I think this should rather be a warning, otherwise the user might modify something they didn't want to modify. Would that make sense? |
I think adding a message that states wether it was the primary or secondary that the mask, size, or whatever has been applied to could help, but this change instead requires the user to explicitely declare that they want to modify the secondary which removes any ambiguity. Unless I misunderstood your concern? |
My concern is that explicitly wanting to change a secondary brush changes the primary brush. But I also see how only preventing that could be confusing. I guess a message clarifying which brush was changed would be enough then. |
|
Please take a moment and address the merge conflicts of your pull request. Thanks! |
Overview
I was asked recently what the
-hflag did in various tool util commands as the existing description of "Whether the offhand should be considered or not" is unclear to others and myself.Upon using it, and from the feedback of others trying to use it too, it felt inconsistent in when it would change the primary or secondary brush.
From reading the code it appears the intent is for tool util commands to change the last used brush when a primary & secondary are set, with the flag inverting which one it impacts.
I think this isn't very intuitive and believe it would make more sense to just have the flag set to change the secondary brush, otherwise the primary.
Description
This changes the
-hflag to always change the value for the secondary brush, if one is set. If no flag is used, or no secondary brush is set, then it will just change the primary brush.I have changed the help page info to make this clear.
Additionally I have added the same flag functionality to the
/size//br sizecommand.Lastly, some minor tidying of the unused editSession import, and the inconsistent tool/brushTool/bt naming.
Hopefully this change makes sense, I'd be curious to hear if myself and those I spoke to are in the minority when it comes to thinking the existing method is unclear and hard to use.