Summary: | EDITING: Undo of formula changes doesn't show affected cells | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Thomas Lendo <thomas.lendo> |
Component: | Calc | Assignee: | Dennis Francis <dennisfrancis.in> |
Status: | VERIFIED FIXED | ||
Severity: | minor | CC: | ilmari.lauhakangas |
Priority: | medium | Keywords: | bibisectRequest, regression |
Version: | Inherited From OOo | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=112358 | ||
Whiteboard: | target:6.0.0 target:5.4.2 target:7.4.0 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 105948 |
Description
Thomas Lendo
2017-05-19 09:28:11 UTC
The only change where it does not jump in 3.6 is B1, so let's add a bibisectRequest to find out what broke C1 jump. Arch Linux 64-bit, KDE Plasma 5 Version: 5.5.0.0.alpha0+ Build ID: c855400e9686ddd8bcba5691393f839f6f52c966 CPU threads: 8; OS: Linux 4.11; UI render: default; VCL: kde4; Locale: fi-FI (fi_FI.UTF-8); Calc: group Built on June 2nd 2017 Arch Linux 64-bit Version 3.6.7.2 (Build ID: e183d5b) Dennis Francis committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f2c29539d52095ea7b914b20ef7f564469d2aa96 tdf#107952: Move cursor to changed cell after undo/redo in ScUndoSetCell It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Dennis Francis committed a patch related to this issue. It has been pushed to "libreoffice-5-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f69cdbe0252a2c70a2a7db1157b06a3041bd39ad&h=libreoffice-5-4 tdf#107952: Move cursor to changed cell after undo/redo in ScUndoSetCell It will be available in 5.4.2. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Dennis, thank you very much for your patch! Undo of C1 changes is now visible to the user. Undo of B1 changes is not covered by the patch. Should I open a new bug for that? Changing summary to "EDITING: Undo of formula changes doesn't show affected cells" to reflect that the patch is only for formula changes but not for styles formatting changes (see Bug 112358 - EDITING: Undo of style formatting doesn't show affected cells). Set VERIFIED FIXED for changes in cell C1 as described in comment 0. Thank you for your work, Dennis! Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/8a9cd6c9fffc89cf00337e7caeca057567867a67 tdf#107952: sc_uicalc: Add unittest It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |