Can we please have a proper design task, leading to a proper design (both on UI/UX and technical aspects) signed of by the relevant team(s) first? A patch review is not the place to have…
We already have many (many) bug reports about that generic issue (transitions between undo steps of different types), and this is not a design task at all. So will archive it for now.
Note that…
No activity for more than a week. As per the tracker policy we assume the issue is gone and can be closed.
Thanks again for the report. If the problem persists please open a new report with…
Patch LGTM, but the name of the commit is not valid, it should not hide a change to a generic API behind a issue fix.
If we wanted that behavior, indeed non-currently-used pointers should be cleared when the source is switched. That way info would not be stored in file etc.
But as long as the reference to the…
The logic in this code is also wrong, these two pointers should be expanded, regardless of the source setting.