Bug 143527 - Impress slide transition sounds do not loop (64-bit Win only)
Summary: Impress slide transition sounds do not loop (64-bit Win only)
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
5.2.0.4 release
Hardware: x86-64 (AMD64) Windows (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: notBibisectable, regression
Depends on:
Blocks: Slide-Transitions
  Show dependency treegraph
 
Reported: 2021-07-24 11:16 UTC by Chris Ford
Modified: 2023-10-10 03:14 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments
Simple reproducer file per instructions (12.28 KB, application/vnd.oasis.opendocument.presentation)
2021-08-05 14:05 UTC, NISZ LibreOffice Team
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Ford 2021-07-24 11:16:47 UTC
Description:
Slide transition sound is not looped, even with the 'Loop until next sound' check box checked. Adding the sounds and playing the video simply plays the audio file once and stops. I extended the duration of the transition to see if that would play the audio more than once but no change. I have tried this with numerous audio tracks but the outcome is always the same. The 'loop until next sound' option does not appear to work.

Steps to Reproduce:
1. Open impress and blank presentation. 
2. In the Slide transition menu, specify a transition and add a sound in the 'Modify Transition' section.
3. Tick the 'Loop until next sound' 
4. Play presentation. Sound only plays once.

Actual Results:
Sound only plays once and does not repeat.

Expected Results:
Sound should repeat.


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.1.4.2 (x64) / LibreOffice Community
Build ID: a529a4fab45b75fefc5b6226684193eb000654f6
CPU threads: 8; OS: Windows 10.0 Build 19042; UI render: Skia/Raster; VCL: win
Locale: en-GB (en_GB); UI: en-GB
Calc: CL
Comment 1 Timur 2021-07-26 10:04:24 UTC
Already seen on ticking 'Loop until next sound', it should play a few times. 
No repro 5.3, repro 5.4 and 7.3+ in Windows only. Regression.
Comment 2 Timur 2021-07-26 10:05:18 UTC
*** Bug 131809 has been marked as a duplicate of this bug. ***
Comment 3 NISZ LibreOffice Team 2021-08-05 13:55:41 UTC
(In reply to Timur from comment #1)
> Already seen on ticking 'Loop until next sound', it should play a few times. 
> No repro 5.3, repro 5.4 and 7.3+ in Windows only. Regression.

I can hear looping effect in bibisect-win32-6.3 master, but not anymore in bibisect-win64-6.4 oldest. Are you sure about 5.3 and 5.4 ?

Also, what could have changed in bibisect repos between 6.3 latest and 6.4 oldest? Apart from the repos themselves being 32 vs 64 bit.
Comment 4 NISZ LibreOffice Team 2021-08-05 14:04:27 UTC
No repro with SI-GUI version x86:

Verzió: 6.4.0.3 (x86)
Build az.: b0a288ab3d2d4774cb44b62f04d5d28733ac6df8
CPU szálak: 4; OS: Windows 10.0 Build 18363; Felületmegjelenítés: alapértelmezett; VCL: win; 
Területi beállítások: hu-HU (hu_HU); Felület nyelve: hu-HU
Calc: threaded

and TB39 nightly:

Version: 7.3.0.0.alpha0+ (x86) / LibreOffice Community
Build ID: 41f1d389d4b513a7f616542d00fd58268ab5ea07
CPU threads: 4; OS: Windows 10.0 Build 18363; UI render: Skia/Raster; VCL: win
Locale: hu-HU (hu_HU); UI: en-US
Calc: CL
Comment 5 NISZ LibreOffice Team 2021-08-05 14:05:09 UTC
Created attachment 174105 [details]
Simple reproducer file per instructions
Comment 6 Aron Budea 2021-08-06 02:07:01 UTC
Based on comment 4, this is fine in current 32-bit Windows builds.
I can confirm the bug already exists in LO 5.2.0.4 64-bit Win build, but is fine in a 5.0.0.5 64-bit Win build.
The oldest 64-bit bibisect repo is 6.4, therefore this is not bibisectable.

I don't see the rationale behind the high/major importance either, resetting it to medium/normal, if anyone wants to raise it, please give a reason.
Comment 7 Timur 2021-08-06 07:27:51 UTC
Good catch for 64-bit. No repro 5.1 alpha 64-bit Win so properly mark from 5.2.

As for importance, 2 reasons:
- this is a general LO feature, which doesn't work for any user (and all are on 64-bit now), not a problem with single document, so may be considered major, and more important:
- this is a regression, and all regressions should have one step higher importance, i.e. high and not medium.
Comment 8 Aron Budea 2021-08-06 16:46:44 UTC
(In reply to Timur from comment #7)
> As for importance, 2 reasons:
> - this is a general LO feature, which doesn't work for any user (and all are
> on 64-bit now), not a problem with single document, so may be considered
> major, and more important:
> - this is a regression, and all regressions should have one step higher
> importance, i.e. high and not medium.
Thanks for giving a reasoning, I agree with having one step higher priority for regressions in general, at the same time the importance should probably be unaffected by this. I'd also think looping the sound is a minor extension of being able to play sounds, which works (there's probably a reason why the regression was first reported 5 years after it was introduced). However, if there's consensus the priority should be higher here, please don't mind me, and go ahead.
Comment 9 Julien Nabet 2021-10-09 11:29:43 UTC
On pc Debian x86-64 with master sources updated today, I don't reproduce this with gtk3, as expected.

But with gen rendering, I could reproduce this. However the duration is taken into account and each time I change duration, the preview is triggered and I can hear the sound (again only once even if loop is enabled) + the transition.

On Windows 10 with master sources updated today, I could reproduce this and in addition, duration isn't taken into account + when I change duration, the preview isn't triggered.
Comment 10 Julien Nabet 2021-10-09 11:32:23 UTC
With kf5 rendering, changing duration triggers the preview, the sound is indeed repeated but the repeat isn't constant, eg: you can hear the sound once then 2 seconds without anything, then 3 times the sound in a row.
Comment 11 Julien Nabet 2021-10-09 11:43:50 UTC
Caolán: another case of different behaviour according the rendering.
Can we consider then it's vcl related or would it be event mechanism which differs between gtk, kde, gen/Windows ?
Comment 12 Julien Nabet 2021-10-09 11:48:57 UTC
Related files:
- sd/uiconfig/simpress/ui/slidetransitionspanel.ui
- sd/source/ui/animations/SlideTransitionPane.cxx
Comment 13 QA Administrators 2023-10-10 03:14:38 UTC
Dear Chris Ford,

To make sure we're focusing on the bugs that affect our users today, LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed bugs which have not been touched for over a year.

There have been thousands of bug fixes and commits since anyone checked on this bug report. During that time, it's possible that the bug has been fixed, or the details of the problem have changed. We'd really appreciate your help in getting confirmation that the bug is still present.

If you have time, please do the following:

Test to see if the bug is still present with the latest version of LibreOffice from https://www.libreoffice.org/download/

If the bug is present, please leave a comment that includes the information from Help - About LibreOffice.
 
If the bug is NOT present, please set the bug's Status field to RESOLVED-WORKSFORME and leave a comment that includes the information from Help - About LibreOffice.

Please DO NOT

Update the version field
Reply via email (please reply directly on the bug tracker)
Set the bug's Status field to RESOLVED - FIXED (this status has a particular meaning that is not 
appropriate in this case)


If you want to do more to help you can test to see if your issue is a REGRESSION. To do so:
1. Download and install oldest version of LibreOffice (usually 3.3 unless your bug pertains to a feature added after 3.3) from https://downloadarchive.documentfoundation.org/libreoffice/old/

2. Test your bug
3. Leave a comment with your results.
4a. If the bug was present with 3.3 - set version to 'inherited from OOo';
4b. If the bug was not present in 3.3 - add 'regression' to keyword


Feel free to come ask questions or to say hello in our QA chat: https://web.libera.chat/?settings=#libreoffice-qa

Thank you for helping us make LibreOffice even better for everyone!

Warm Regards,
QA Team

MassPing-UntouchedBug