Bug 119146

Summary: Fly in Words with non-animated background show entire paragraph in addition
Product: LibreOffice Reporter: Regina Henschel <rb.henschel>
Component: ImpressAssignee: Mike Kaganski <mikekaganski>
Status: VERIFIED FIXED    
Severity: normal CC: ilmari.lauhakangas, mikekaganski, telesto
Priority: medium Keywords: bibisected, bisected, regression
Version: 6.1.0.2 rc   
Hardware: All   
OS: All   
Whiteboard: target:6.2.0 target:6.1.2
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 109362    
Attachments: Test FlyIn animation with Word-by-Word

Description Regina Henschel 2018-08-07 17:19:51 UTC
Created attachment 144016 [details]
Test FlyIn animation with Word-by-Word

Open attached document and play it. You should see an orange rectangle. On click words should fly in from top to this rectangle.

In version 6.1 not only the words fly in but the entire paragraph flies in in addition. Use the document in LO6.0 to see how it should work.

Bug in Version: 6.1.0.2 (x64)
Build ID: b3972dcf1284967612d5ee04fea9d15bcf0cc106
CPU threads: 8; OS: Windows 10.0; UI render: default; 
Locale: de-DE (en_US); Calc: CL

OK in Version: 6.0.4.2 (x64)
Build-ID: 9b0d9b32d5dcda91d2f1a96dc04c645c450872bf
CPU-Threads: 8; BS: Windows 10.0; UI-Render: Standard; 
Gebietsschema: de-DE (en_US); Calc: CL
Comment 1 Buovjaga 2018-09-02 18:00:53 UTC
Bibisected with win32-6.2 to https://cgit.freedesktop.org/libreoffice/core/commit/?id=4600b07c1d787f959618d9ecf54161e4ea4ffa61

tdf#102195: don't exclude everything between first and last subset
When subsets are non-contiguous, we need to include the parts between
subsets.

Adding Cc: to Mike Kaganski

Also repro on Linux.
Comment 2 Mike Kaganski 2018-09-03 16:45:47 UTC
https://gerrit.libreoffice.org/59948
Comment 3 Mike Kaganski 2018-09-03 16:54:24 UTC
(In reply to Buovjaga from bug 102195 comment 1)
> We need 2 confs per year to discover all these sneaky ones..

Your say The Truth :-)
Comment 4 Commit Notification 2018-09-03 20:26:31 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=a8aaffa6ea0f0fd8cf1d22883173df64a93e3322

tdf#119146: fix populating maCurrentSubsets

It will be available in 6.2.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.
Comment 5 Mike Kaganski 2018-09-04 04:10:18 UTC
https://gerrit.libreoffice.org/59952 is a backport to 6-1.
Comment 6 Commit Notification 2018-09-04 07:59:57 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "libreoffice-6-1":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=328646778fcf7242863f0fe20f33bd99d3386c63&h=libreoffice-6-1

tdf#119146: fix populating maCurrentSubsets

It will be available in 6.1.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.
Comment 7 Buovjaga 2018-09-04 18:08:39 UTC
*Snif* - it's such a beautiful fly in again. Thanks!

Arch Linux 64-bit
Version: 6.2.0.0.alpha0+
Build ID: bf8fbbaa683ef7eef5f9587b60486f622b50cb80
CPU threads: 8; OS: Linux 4.18; UI render: default; VCL: gtk3_kde5; 
Locale: fi-FI (fi_FI.UTF-8); Calc: threaded
Built on September 4th 2018