Bug 119144 - unsuitable field value appearance for "delay between characters" setting in animation
Summary: unsuitable field value appearance for "delay between characters" setting in a...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
6.2.0.0.alpha0+
Hardware: x86-64 (AMD64) Windows (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Impress-UX Custom-Animations
  Show dependency treegraph
 
Reported: 2018-08-07 15:13 UTC by Regina Henschel
Modified: 2019-01-11 10:03 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
File to test UI for animation (13.14 KB, application/vnd.oasis.opendocument.presentation)
2018-08-07 15:13 UTC, Regina Henschel
Details
Test FlyIn animation with Word-by-Word (14.53 KB, application/vnd.oasis.opendocument.presentation)
2018-08-07 17:55 UTC, Regina Henschel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Regina Henschel 2018-08-07 15:13:59 UTC
Created attachment 144014 [details]
File to test UI for animation

The text in the attached file has an animation "Appear" with setting "Word by word". Open the advanced animation settings. You see a label "delay between characters". The shown value is "34.567%" for me with German local with dot as thousand separator.

But actually
*  the value is not a percent [percent of what should it be?]
*  the value is not "between characters" but "between words"
*  the value is written to file as PT00H00M00.34567S. That means it is in fact 0.34567 seconds.

Expected behavior:
The field acts as a metric field for decimal in unit 'seconds'.
The unit of the field is mentioned.
The label adapts to the chosen affect.

If you newly create such animation, the default value is "10%". That is unsuitable, because it will result in a duration of 0.0001 seconds. The complete animation will play nearly at once. The user will think, that the animation does not work and he gets no information about the needed magnitude.
Comment 1 Regina Henschel 2018-08-07 17:55:12 UTC
Created attachment 144017 [details]
Test FlyIn animation with Word-by-Word

I have now tested animation Typ "Fly In". This animation type has a field "Duration". And in this case a value of "10%" for "delay between characters" makes more sense. It refers to the field "Duration". So a setting of "Duration"=3sec and "Delay between characters"=10% results in anim:iterate-interval="PT00H00M00.3S" in the file. The actual duration of the effect is then "iterate-interval"*"number of parts".

In this case, the user does not know the reference of the percent value. And the actual duration of the effect is different from "Duration". So still a bad UI.
Comment 2 Mark Hung 2018-08-22 11:03:24 UTC
As shown here

https://opengrok.libreoffice.org/xref/core/sd/source/ui/animations/CustomAnimationPane.cxx#1159

The number in percentage of duration if duration is defined, and is in 10 milliseconds if duration is not defined.

And here
https://opengrok.libreoffice.org/xref/core/sd/source/ui/animations/CustomAnimationPane.cxx#1291

not sure what will happen when duration is not defined. 
So there is some problem when display the iterate interval.
Comment 3 Roman Kuznetsov 2019-01-10 08:30:40 UTC
Heiko, I think need some UX love for this. What do you think?
Comment 4 Heiko Tietze 2019-01-11 10:03:07 UTC
(In reply to Roman Kuznetsov from comment #3)
> Heiko, I think need some UX love for this. What do you think?

Ordinary bug, IMHO. As Regina expects "The field acts as a metric field for decimal in unit 'seconds'" (maybe milliseconds). "All at once" disables the field and letter or word take the value. Default is today 10%, which should be 0.25s per word and 0.1s per letter, or the like.

Btw, +/- don't work for me and I cannot enter a value. 

Version: 6.1.4.2
Build ID: 6.1.4-1
CPU threads: 8; OS: Linux 4.20; UI render: default; VCL: gtk3; 
Locale: de-DE (en_US.UTF-8); Calc: group threaded