Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
[ui] rename the min/max widget group box title
Since there is no more loading involved through the min/max widget, rename it to: "Min /max values settings"
- Loading branch information
407991eThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rouault , I was wondering whether the "Updated canvas" label was a bit obscure for the newcomers. Do you think "Dynamic canvas" could a better naming for the function? IMHO it does a better job at reflecting what it does.
407991eThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wasn't satisfied with the Updated canvas label either. I think I also considered Refreshed canvas. But dynamic sounds better
407991eThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rouault , do you feel like doing the bit of renaming required (i.e. the QgsRasterMinMaxOrigin::UpdatedCanvas, etc.)? If not, I can commit to it.
407991eThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rouault , an extra bonus to "Dynamic canvas", it'll help people leaving ArcGIS behind to link that with ESRI's equivalent (if memory serves well, it's called dynamic range adjustment).