Merge lp:~om26er/mediaplayer-app/disable_settings_button into lp:mediaplayer-app

Proposed by Omer Akram
Status: Merged
Merged at revision: 162
Proposed branch: lp:~om26er/mediaplayer-app/disable_settings_button
Merge into: lp:mediaplayer-app
Diff against target: 12 lines (+2/-0)
1 file modified
src/qml/player/Controls.qml (+2/-0)
To merge this branch: bzr merge lp:~om26er/mediaplayer-app/disable_settings_button
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Ugo Riboni (community) Approve
Review via email: mp+190904@code.launchpad.net

Commit message

disable the settings button as there are no settings

Description of the change

disable the settings button as there are no settings

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ugo Riboni (uriboni) wrote :

Code looks fine, feel free to top-approve when jenkins is ok with it

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/qml/player/Controls.qml'
2--- src/qml/player/Controls.qml 2013-09-30 21:48:25 +0000
3+++ src/qml/player/Controls.qml 2013-10-14 09:58:59 +0000
4@@ -254,6 +254,8 @@
5
6 width: units.gu(9)
7 height: units.gu(3)
8+ enabled: false
9+ opacity: 0.2
10
11 onClicked: settingsClicked()
12 }

Subscribers

People subscribed via source and target branches