spreadsheet cell with several words, e.g. "Aap Noot Mies" Enter edit mode, copy "Mies", push HOME, then Ctrl+V Result (with | representing cursor position) "|MiesAap Noot " Expected "Mies|Aap Noot " Noticed in Versie: 6.0.0.1 Build ID: d2bec56d7865f05a1003dc88449f2b0fdd85309a CPU-threads: 4; Besturingssysteem: Linux 4.13; UI-render: standaard; VCL: gtk2; Locale: nl-NL (nl_NL.UTF-8); Calc: group
Works OK in 5.4.4.2 > regression
Regression introduced by: author Varun Dhall <varun.dhall@studentpartner.com> 2017-07-13 00:36:46 +0530 committer Michael Stahl <mstahl@redhat.com> 2017-07-12 23:28:06 +0200 commit e333183d4390da0b17a55f214e2b953dbb0a8883 (patch) tree 680493fcca8e2a8239838db841cd623c2cedf1d3 parent b392208dde22edbcba36fe516ac114284a111728 (diff) EditEngine: Making ODF Format Default for Copy/Paste Bisected with: bibisect-linux64-6.0 Adding Cc: to Varun Dhall
Noticing the same problem when editing a text shape in Draw Version: 6.1.0.0.alpha0+ Build ID: 7a6df88f2579c4fcfa81eb27f6db9e8e21c9a6f8 CPU threads: 4; OS: Linux 4.13; UI render: default; VCL: gtk2; TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2018-01-14_22:46:48 Locale: nl-NL (nl_NL.UTF-8); Calc: group threaded
Michael, one for you ?
Commit mentioned in comment 2 is not the real cause, rather it switches the default clipboard format to the new implementation. Note that this does not happen for "Unformatted text" (i.e. selecting that after Shift+Ctrl+V PasteSpecial) but also happens for "Formatted text [RTF]". Likely that path forgets to update the cursor position / selection with adding the length of the newly inserted text portion.
*** Bug 115536 has been marked as a duplicate of this bug. ***
(In reply to Eike Rathke from comment #5) > Commit mentioned in comment 2 is not the real cause, rather it switches the > default clipboard format to the new implementation. > > Note that this does not happen for "Unformatted text" (i.e. selecting that > after Shift+Ctrl+V PasteSpecial) but also happens for "Formatted text > [RTF]". Likely that path forgets to update the cursor position / selection > with adding the length of the newly inserted text portion. @Eike, Do you think you could take a look at it at some point ?
Well, actually I hoped for Varun fixing this. Varun?
Apparently not.. taking.
*** Bug 115792 has been marked as a duplicate of this bug. ***
Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a44e62b82c7598471da9a5254953067613806337 Resolves: tdf#114675 return new EditPaM of insert position from SvxReadXML() It will be available in 6.1.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.
Pending review https://gerrit.libreoffice.org/50646 for 6-0
Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-6-0": http://cgit.freedesktop.org/libreoffice/core/commit/?id=2ebda414bfc5b4961ddb41e5512de215476a31a2&h=libreoffice-6-0 Resolves: tdf#114675 return new EditPaM of insert position from SvxReadXML() It will be available in 6.0.3. 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.
Tested with Version: 6.1.0.0.alpha0+ (x64) Build ID: 751191ed2d7d6af6eddc3d738e8c45b0a2ab2572 CPU threads: 4; OS: Windows 10.0; UI render: GL; TinderBox: Win-x86_64@42, Branch:master, Time: 2018-03-22_00:36:35 Locale: de-DE (de_DE); Calc: CL and with Version: 6.0.4.0.0+ (x64) Build ID: deb5db7bf6d45338c9b6f6f4a1d62fba168b7a85 CPU threads: 4; OS: Windows 10.0; UI render: GL; TinderBox: Win-x86_64@42, Branch:libreoffice-6-0, Time: 2018-03-22_03:08:26 Locale: de-DE (de_DE); Calc: CL Both tests were succesful: Cursor was set correctly behind the pasted text.