Skip to content
GitLab
Menu
Projects
Groups
Snippets
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
Menu
Open sidebar
PIM
KAlarm
Commits
7a6348bd
Commit
7a6348bd
authored
Apr 25, 2022
by
David Jarvie
Browse files
Fix Stop Play button not working for display alarm with audio file
parent
30d5a943
Pipeline
#168432
passed with stage
in 2 minutes and 55 seconds
Changes
3
Pipelines
1
Hide whitespace changes
Inline
Side-by-side
Changelog
View file @
7a6348bd
...
...
@@ -4,6 +4,7 @@ KAlarm Change Log
* Fix checkboxes being disabled in Preferences dialogue.
* Fix time spin boxes being displayed in the wrong position.
* Fix crash when Try clicked to stop audio alarm with fade [KDE Bug 452962]
* Fix Stop Play button not working for display alarm with audio file.
=== Version 3.4.0 (KDE Gear 22.04) --- 6 April 2022 ===
* Allow calendars and date picker to be shown together in side panel [KDE Bug 440250]
...
...
src/messagedisplayhelper.cpp
View file @
7a6348bd
...
...
@@ -883,7 +883,7 @@ void MessageDisplayHelper::startAudio()
connect
(
audioPlayer
,
&
AudioPlayer
::
readyToPlay
,
this
,
&
MessageDisplayHelper
::
playReady
);
connect
(
audioThread
,
&
QThread
::
finished
,
this
,
&
MessageDisplayHelper
::
playFinished
);
if
(
mSilenceButton
)
connect
(
mSilenceButton
,
&
QAbstractButton
::
clicked
,
audioThread
,
&
QThread
::
quit
);
connect
(
mSilenceButton
,
&
QAbstractButton
::
clicked
,
this
,
&
MessageDisplayHelper
::
stopAudioPlay
);
// Notify after creating mAudioPlayer, so that isAudioPlaying() will
// return the correct value.
...
...
src/messagedisplayhelper.h
View file @
7a6348bd
...
...
@@ -130,6 +130,7 @@ private Q_SLOTS:
void
slotSpeak
();
void
audioTerminating
();
void
startAudio
();
void
stopAudioPlay
()
{
stopAudio
();
}
void
playReady
();
void
playFinished
();
void
slotSetRemainingTextDay
()
{
setRemainingTextDay
(
true
);
}
...
...
Write
Preview
Supports
Markdown
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment