Sometimes is required to reset the thickness or the opacity of the strokes. Actually this was done using a modifier, but this operators solves this.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- temp-gpencil-normalize (branched from master)
- Build Status
Buildable 14880 Build 14880: arc lint + arc unit
Event Timeline
Comment Actions
Just a few comments for code readability and clarity
| source/blender/editors/gpencil/gpencil_edit.c | ||
|---|---|---|
| 5392 | Because we are checking against the value of an enum here, I would rather use the explicit else if (mode == GP_NORMALIZE_OPACITY) or use a switch statement. Just so that we know what mode is used when the current else part is run. Makes the code more readable :) | |
| 5434 | Would rather use and early continue here so that the rest is not indented. | |
| 5491 | Maybe it's just me, but I would be a bit more explicit with the tooltips. I don't know what "Normalize Thickness" or "Normalize Opacity" means. | |
| source/blender/editors/gpencil/gpencil_edit.c | ||
|---|---|---|
| 5495–5503 | I'd suggest for the tooltip: "Normalizes the stroke thickness by making all points use the same thickness value" "Normalizes the stroke opacity by making all points use the same opacity value" | |