Objective
Improve the quality of the brush spacing with respect to velocity.
Related Links
- Branches: Artbox and feature-dynamic-brush-spacing
Design Revisions
| Revision | Current Design | Issues | Changes |
|---|---|---|---|
| 1. Remove Spacing Limits | Spacing is set to 200% for fast | Spacing does not respect the Paint Tool slider settings | Spacing respects the slider values |
Notes
This feature branch was a proposed solution to https://gitlab.gnome.org/GNOME/gimp/-/issues/1863
Without Feature: Top is a slow stroke, middle line fast, lower is slow to fast.
With Feature: Top is a slow stroke, middle line fast, lower is slow to fast.
MR Description
This MR refines the calculation of dynamic brush spacing, removing unnecessary scaling and simplifying the logic.
Changes
- Simplified Dynamic Spacing Calculation:
- Removed the formula that scaled spacing up to 200%.
- Now, dynamic spacing is directly multiplied by
core->spacing, ensuring a simpler and more predictable behavior. - The minimum spacing is clamped to
EPSILONto prevent it from becoming too small.
Benefit
- The new approach provides cleaner, more predictable spacing behavior, adhering more closely to user-defined values without arbitrary limits.