Merge lp:~lemonboy/noise/noise-privacy-mode into lp:~elementary-apps/noise/trunk
Proposed by
The Lemon Man
Status: | Merged |
---|---|
Approved by: | Corentin Noël |
Approved revision: | 1846 |
Merged at revision: | 1848 |
Proposed branch: | lp:~lemonboy/noise/noise-privacy-mode |
Merge into: | lp:~elementary-apps/noise/trunk |
Diff against target: |
140 lines (+37/-22) 4 files modified
core/Settings.vala (+10/-5) src/LibraryWindow.vala (+11/-7) src/PlaybackManager.vala (+5/-2) src/Views/ListView/Lists/GenericList.vala (+11/-8) |
To merge this branch: | bzr merge lp:~lemonboy/noise/noise-privacy-mode |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
kay van der Zander (community) | Approve | ||
Corentin Noël | Approve | ||
Review via email: mp+274444@code.launchpad.net |
Commit message
Respect Privacy Mode
To post a comment you must log in.
Hey nice work,
Please place the function privacy_ mode_enabled inside a class. Logical would be the main class in settings.vala.
You get main_settings. privacy_ mode_enabled( ) in the if statement in the on_quit function.
Next time no functions outside a class. In case of static it is allowed :) but not needed in this case.