Bug 127663 - Undo font resize needs multiple undo actions
Summary: Undo font resize needs multiple undo actions
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: iOS Editor (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other iOS
: medium normal
Assignee: Jan Holesovsky
URL:
Whiteboard: target:6.4.0
Keywords:
Depends on:
Blocks:
 
Reported: 2019-09-20 13:18 UTC by Nicolas Christener
Modified: 2019-09-26 09:36 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:

Attachments
Video demonstrating the issue (658.22 KB, video/mp4)
2019-09-20 13:21 UTC, Nicolas Christener 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Christener 2019-09-20 13:18:43 UTC 
Description:
Resizing the font size of some text works well but undoing such a change needs multiple undo actions.

Steps to Reproduce:
1. Create a document with the iOS app
2. Enter some text and resize the font size e.g. from 14pt to 28ot
3. Tab undo once

Actual Results:
Nothing visibily happens.

Expected Results:
The font size should go back to the last size.


Reproducible: Always


User Profile Reset: No



Additional Info:
If I tap some more time on the undo button it will work at some point.

In my tests this seems to happen after the 8th try.
Comment 1 Nicolas Christener 2019-09-20 13:21:33 UTC 
Created attachment 154322 [details]
Video demonstrating the issue
Comment 3 Commit Notification 2019-09-26 07:46:38 UTC 
Jan Holesovsky committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/online/commit/311e93fb90cd9480368eeabf2e5adc264af30eed

tdf#127663: Don't emit 'uno .uno:FontHeight ...' multiple times.
Comment 4 Jan Holesovsky 2019-09-26 09:36:46 UTC 
Fixed.