Node tool: handle size of selected nodes too big and sticky (rev >= 14761)
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Inkscape |
Low
|
Antonio Ospite | ||
| | 0.92.x |
Low
|
Antonio Ospite | ||
Bug Description
The handles of nodes as displayed in the node tool context usually enlarge slightly when selected, and revert to the original handle size (according to the user's setting) once deselected.
After the refactoring in rev 14761 [1] (to fix UX regression for pen/pencil start and end node handles), the handles in the node tool context grow on selection by a too large amount, and do not revert to the original size on deselection of the node: the path has to be deselected as a whole, and re-selected again to display with the expected handle size.
Attached are screen recordings of two different recent builds:
- Inkscape 0.91+devel r14760 (ok)
- Inkscape 0.91+devel r14770 (not ok)
[1] Related commit:
https:/
| su_v (suv-lp) wrote : | #1 |
| su_v (suv-lp) wrote : | #2 |
Screen recording r14770 (not ok)
| Changed in inkscape: | |
| assignee: | nobody → Martin Owens (doctormo) |
| status: | New → Confirmed |
| su_v (suv-lp) wrote : | #3 |
@JazzyNico - could you please target to milestone 0.92? This regression can quickly turn node-editing sessions of more complex paths into a rather cumbersome activity (the mix of handle sizes after having selected and de-selected nodes is confusing, visually, at least to me).
| Changed in inkscape: | |
| milestone: | none → 0.92 |
| status: | Confirmed → Triaged |
| Changed in inkscape: | |
| importance: | Undecided → Low |
| jazzynico (jazzynico) wrote : | #4 |
Any chance it could be fixed for 0.92.1?
If not, please retarget to 0.93. Thanks!
| Changed in inkscape: | |
| milestone: | 0.92 → 0.92.1 |
| Changed in inkscape: | |
| milestone: | 0.92.1 → 0.93 |
| Antonio Ospite (ospite) wrote : | #5 |
I took a look and the problem is that the changes from[1] introduce a bug in ControlManagerI
The problem is that the previos code was changin targetSize directly while the new code sets item->ctrlResize = 2 conditionally but then does not use this value for the actual size, but the fixed _resize value.
The attached patch fixes the issue.
If the patch is OK let me know if you are going to apply it with git-am or if you prefer a merge request on gitlab.
Ciao,
Antonio
| Mc (mc...) wrote : | #6 |
| Changed in inkscape: | |
| assignee: | Martin Owens (doctormo) → Antonio Ospite (ospite) |
| status: | Triaged → Fix Committed |
| Qantas94Heavy (qantas94heavy) wrote : | #7 |
Fix released in 0.92.3.
| Changed in inkscape: | |
| status: | Fix Committed → Fix Released |
Screen recording r14760 (ok)