Merge lp://qastaging/~mc.../inkscape/mc-selection-transform into lp://qastaging/~inkscape.dev/inkscape/trunk
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://qastaging/~mc.../inkscape/mc-selection-transform |
| Merge into: | lp://qastaging/~inkscape.dev/inkscape/trunk |
| Diff against target: |
3772 lines (+882/-872) 3 files modified
src/selection-chemistry.cpp (+865/-872) src/selection.cpp (+7/-0) src/selection.h (+10/-0) |
| To merge this branch: | bzr merge lp://qastaging/~mc.../inkscape/mc-selection-transform |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jabiertxof | code | Approve | |
|
Review via email:
|
|||
Description of the change
Refactoring of sp_selection_
| Mc (mc...) wrote | # |
Unmerged revisions
- 14347. By Mc
-
more clang-format because i can never get the spacing right
- 14346. By Mc
-
final comments and refactor
- 14345. By Mc
-
refactor + fixed linked offsets
- 14344. By Mc
-
I was not /that/ far yesterday at 4am. Ready to be tested !
- 14343. By Mc
-
works better than previously, but there is still some strange behavior
- 14342. By Mc
-
Wait a minute, is that really working ? Have those several hours with paper and pen, filling pages with matrix multiplications, not been in vain ?
Just kidding, still WIP. make a clone of clone, move the original and first clone, the second moves unexpectedly.
But still, the fact that the first clone behaves right is already pretty cool !
- 14341. By Mc
-
I'm so confused right now.
- 14340. By Mc
-
does not break anything yet. I still have to fix the case where i resize a (original+(clone in a group)) selection
- 14339. By Mc
-
clang-format src/selection-
chemistry. cpp - 14338. By Mc
-
add includes_deeply helper function to test if an object is selected, even indirectly
NB: The diff is huge because i made a clang-format to have all the presentation right, the only non-whitespace parts are lines 1297-1651 and 3728+