Merge lp://staging/~josharenson/indicator-session/clarify-options into lp://staging/indicator-session
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://staging/~josharenson/indicator-session/clarify-options |
| Merge into: | lp://staging/indicator-session |
| Diff against target: |
12 lines (+1/-1) 1 file modified
src/service.c (+1/-1) |
| To merge this branch: | bzr merge lp://staging/~josharenson/indicator-session/clarify-options |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Lukáš Tinkl (community) | Needs Information | ||
|
Review via email:
|
|||
Commit message
Show the greeter when "Lock/Siwtch Users" is clicked
This is instead of only showing the lockscreen
Description of the change
Show the greeter when "Lock/Siwtch Users" is clicked
This is instead of only showing the lockscreen. The inactivity timeout and Lock key-combo still trigger the internal lockscreen, but since the lockscreen can't switch users, this patch shows the actual greeter when "Lock/Switch Users" is clicked.
* Ensure the project compiles and the test suite executes without error
The tests segfault, but they do on trunk as well
* Ensure that non-obvious code has comments explaining it
N/A
* If the change works on specific profiles, please include those in the merge description.
N/A
| Josh Arenson (josharenson) wrote | # |
| Josh Arenson (josharenson) wrote | # |
Unmerged revisions
- 480. By Josh Arenson
-
Show the greeter when "Lock/Siwtch Users" is clicked
This is instead of only showing the lockscreen
I'm curious as to why the tests are not running here (segfaulting?) Worth investigating maybe