Bug 149854 - Track Changes: replace of the first letter of a word is not consistent; new character at the left or right of deleted one depending on how (STR comment 8)
Summary: Track Changes: replace of the first letter of a word is not consistent; new c...
Status: RESOLVED DUPLICATE of bug 109272
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.1.0.3 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-04 18:43 UTC by Telesto
Modified: 2023-02-02 09:35 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot of difference between 6.3 and 7.4 (250.21 KB, image/jpeg)
2022-07-06 19:22 UTC, Telesto
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2022-07-04 18:43:34 UTC
Description:
Track Changes: replace of the first letter of a word is not consistent; new character at the left or right of deleted one depending on how

Steps to Reproduce:
1. Open attachment 180483 [details]
2. Select the 's' of six in the first line
3. Type S
4. Select the w of ways on the first line, next to six
5. Press backspace
6. Type W. 

Spin-off from bug 149388

Actual Results:
At step 3 the newly added S is at the right of the deleted s , but the W in case of step 4-5 is at the left of the deleted word 

Expected Results:
Left in both cases as previously


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
Version: 7.4.0.0.alpha1+ (x64) / LibreOffice Community
Build ID: 089c91b1ad16232f130cb50266732509f83c52eb
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: nl-NL
Calc: CL

and in
Version: 7.1.8.0.0+ (x64) / LibreOffice Community
Build ID: a94b58277c7aeaa83ce14347cd0b8f7137969d03
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

not in
Version: 6.3.7.0.0+ (x86)
Build ID: 726535ec30f12697ceccd2f0640d9371a64dc5bd
CPU threads: 4; OS: Windows 6.3; UI render: default; VCL: win; 
Locale: nl-NL (nl_NL); UI-Language: en-US
Calc: CL
Comment 1 Rainer Bielefeld Retired 2022-07-05 13:22:41 UTC
NOT completely reproducible with sample document and Installation of Version: 7.3.3.2 (x64) Build ID: d1d0ea68f081ee2800a922cac8f79445e4603348
CPU threads: 12; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: de-DE |  Calc: threaded | ElementaryTheme | My normal User Profile	

0. Open document
1. If necessary activate Menu ˋEdit → Changes → Trach Changesˊ
2. ˋClick left from word "six" → Select "s" only using mouse  → Type "S"ˊ 
   » "S" inserted right from strikethorugh-"s", caret flashing right from new 
     character
3. ˋClick left from word "way" → Select "w" only using mouse  → <Backspace>
   » Caret flashes RIGHT from strikethrough-"w"
4. Type "W"ˊ
   » "W" inserted right from strikethorugh-"w"

That's all as expected, and reporter's observations not reproduced.

But now close document without saving and redo form step (0)
But in Step (2) click right from "s", rest as per step 2
  » You will not see a difference to step (2)

Continue with step (3), but click RIGHT from "w", rest as described above
  » Caret flashes LEFT from strikethrough-"w"
    That's different to (3) above
Now Type "W"ˊ
   » "W" inserted LEFT from strikethorugh-"w"
     That's different from result above, but of course new character will be
     inserted at Caret position.

additional Info
---------------
a) IMHO everything is correct here. After <Backspace> a newly inserted 
   character will be inserted at the caret position
b) As expected I also can't see a regression here. In my tests everything 
   worked same in 6. and 7.
   I think Telesto simply accidentally changed highlighting direction 
   for "w" hen he switched from 7. to 6. during his tests.

@Telesto?
   used the other
Comment 2 Telesto 2022-07-05 18:36:45 UTC
Well I obviously forgot so detail at step 4. This about difference between overwrite, compared to deleting by backspace & typing something else

1. Open attachment 180483 [details]
2. Select the 's' of six in the first line
3. Type S
4. Place the cursor after w of "ways"  on the first line, next to six
5. Press backspace
6. Type W.
Comment 3 Rainer Bielefeld Retired 2022-07-05 19:16:49 UTC
Indeed, there is a relation to "Bug 149873 - Cursor doesn't move the left after pressing backspace with track changes enabled in certain consternation"

Currently I am seeing that more than a feature than as a bug, but that mighbeseen different from a different POV.
Comment 4 Rainer Bielefeld Retired 2022-07-05 19:17:54 UTC
And I still don't see a regression.
Detailed comparison V6.vs. V7 migt help?!
Comment 5 Telesto 2022-07-05 20:57:19 UTC
(In reply to Rainer Bielefeld Retired from comment #3)
> Currently I am seeing that more than a feature than as a bug, but that
> mighbeseen different from a different POV.

I find it hard to tell, the variation is somewhat annoying; Esthetically and for orientation. No clue what MSO does..

Original reporter at bug 149388

(In reply to Rainer Bielefeld Retired from comment #4)
> And I still don't see a regression.
> Detailed comparison V6.vs. V7 migt help?!

I will re-check it tomorrow.
Comment 6 Telesto 2022-07-06 19:22:44 UTC
Created attachment 181145 [details]
Screenshot of difference between 6.3 and 7.4
Comment 7 Buovjaga 2023-01-23 11:58:26 UTC
Testing this a lot (kind of a wild goose chase as I didn't get it at first), I don't see any bug here. The insertion is done logically based on the cursor position.
Comment 8 Telesto 2023-01-23 13:39:57 UTC
Improved steps
1. Open attachment 180483 [details]
2. Edit -> Track Changes -> Record ON & Show ON
2. Select the 's' of six in the first line
3. Type S -> un-capitalized s at the left
4. Put cursor after 'w' in ways (next to six)
5. Press backspace
6. Type W -> un-capitalized w at the right
7. Put cursor before 't' of that
8. Press Delete 
9. Type T -> un-capitalized t at the right

Overwrite insertion behaves like Delete + Insertion (steps 7-9). I would expect it the behave as 'Backspace + Insertion (steps 4-6)

Why? It was working differently in the paste + taste. I use tend to overwrite text by simply overtyping or backspace/insertion. Less often with delete (but that's me)

The end-result is now hotchpotch. If you use overwrite, the old text being at the left. Using backspace + insertion the old text text at the right. Doesn't make reading correction easier

I principle it shouldn't matter if you did use delete/backspace/overwrite to replace something. With show changes the result should look the same. But that's my point of view.

I get that backspace being different compared to delete technically, however this more about readability from end-user perspective
Comment 9 Telesto 2023-01-23 14:02:57 UTC
Another quirk
1. Open attachment 180483 [details]
2. Edit -> Track Changes -> Record ON & Show ON
3. Put cursor after 'w' in ways (next to six)
4. Press backspace
5. Type W -> un-capitalized w at the right
6. Press CTRL+Z (2x) (don't deselect)
7. Press Backspace
8. Type W -> un-capitalized w at the left
Comment 10 Buovjaga 2023-01-23 16:18:02 UTC
(In reply to Telesto from comment #8)
> Improved steps
> 1. Open attachment 180483 [details]
> 2. Edit -> Track Changes -> Record ON & Show ON
> 2. Select the 's' of six in the first line
> 3. Type S -> un-capitalized s at the left
> 4. Put cursor after 'w' in ways (next to six)
> 5. Press backspace
> 6. Type W -> un-capitalized w at the right
> 7. Put cursor before 't' of that
> 8. Press Delete 
> 9. Type T -> un-capitalized t at the right

No, un-capitalized t at the left

> Overwrite insertion behaves like Delete + Insertion (steps 7-9). I would
> expect it the behave as 'Backspace + Insertion (steps 4-6)
> 
> Why? It was working differently in the paste + taste.

It has always worked like this, tested for example 6.0, 3.5. Can you tell in which version it worked differently?
Comment 11 Heiko Tietze 2023-01-26 10:32:25 UTC
Isn't this a duplicate of bug 109277?
Comment 12 Buovjaga 2023-01-26 11:08:06 UTC
(In reply to Heiko Tietze from comment #11)
> Isn't this a duplicate of bug 109277?

Doesn't seem like a dupe of "Control-drag-drop in Writer doesn’t copy"
Comment 13 Heiko Tietze 2023-01-26 11:40:11 UTC
(In reply to Buovjaga from comment #12)
> Doesn't seem like a dupe of "Control-drag-drop in Writer doesn’t copy"

bug 109272 ;-)
Comment 14 Telesto 2023-01-26 13:22:45 UTC

*** This bug has been marked as a duplicate of bug 109272 ***
Comment 15 Heiko Tietze 2023-02-01 18:53:08 UTC
Better have it together with the still open bug 137972.

*** This bug has been marked as a duplicate of bug 137972 ***
Comment 16 Heiko Tietze 2023-02-02 09:34:36 UTC

*** This bug has been marked as a duplicate of bug 109272 ***