Merge lp:~rsalveti/qtubuntu-camera/add_audio_role into lp:qtubuntu-camera

Proposed by Ricardo Salveti
Status: Merged
Approved by: Jim Hodapp
Approved revision: 108
Merged at revision: 110
Proposed branch: lp:~rsalveti/qtubuntu-camera/add_audio_role
Merge into: lp:qtubuntu-camera
Diff against target: 11 lines (+1/-0)
1 file modified
src/aalimagecapturecontrol.cpp (+1/-0)
To merge this branch: bzr merge lp:~rsalveti/qtubuntu-camera/add_audio_role
Reviewer Review Type Date Requested Status
Jim Hodapp (community) code Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+233761@code.launchpad.net

Commit message

Setting media role for the shutter sound (alert)

Description of the change

Setting media role for the shutter sound (alert)

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Jim Hodapp (jhodapp) wrote :

Looks good, thanks

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/aalimagecapturecontrol.cpp'
2--- src/aalimagecapturecontrol.cpp 2014-07-16 14:37:32 +0000
3+++ src/aalimagecapturecontrol.cpp 2014-09-08 17:11:28 +0000
4@@ -51,6 +51,7 @@
5 {
6 m_galleryPath = QStandardPaths::writableLocation(QStandardPaths::PicturesLocation);
7 m_audioPlayer->setMedia(QUrl::fromLocalFile("/system/media/audio/ui/camera_click.ogg"));
8+ m_audioPlayer->setAudioRole(QMediaPlayer::AlertRole);
9 }
10
11 AalImageCaptureControl::~AalImageCaptureControl()

Subscribers

People subscribed via source and target branches