"start screen saver" does nothing if lock-screen is locked down

Bug #1066825 reported by Victor Zhou on 2012-10-15
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Session Menu
Fix Released
Low
Charles Kerr
indicator-session (Ubuntu)
Low
Charles Kerr

Bug Description

Boot into live-cd
Observe that "Start Screen Saver" action does nothing

Related branches

description: updated
tags: added: rls-q-incoming
Brian Murray (brian-murray) wrote :

This should happen in casper or the desktop package that creates the "start screen saver" button not ubiquity which is the installer.

Changed in ubiquity (Ubuntu):
importance: Undecided → Low
tags: added: rls-q-notfixing
removed: rls-q-incoming
affects: ubiquity (Ubuntu) → casper (Ubuntu)
Dimitri John Ledkov (xnox) wrote :

Actually we do not want to start screen-saver at all, once ubiquity is running. This is part of the design, as a black screen can be confusing. So we actually disable the screensaver when running ubiquity, see bug 951336. So there should be no way to activate screensaver. Can you add a screen / photo where that is visible?

Victor Zhou (victor.zhou) wrote :

to Dmitrijs Ledkovs,
the attachment is the screen shot.

Dimitri John Ledkov (xnox) wrote :

Thanks.... it ought to be removed.

affects: casper (Ubuntu) → ubiquity (Ubuntu)
Changed in ubiquity (Ubuntu):
status: New → Triaged
Dimitri John Ledkov (xnox) wrote :

Hm.... I am actually confused where that item is coming from. The fact that it's not translated, also suggests that it's not under ubiquity's control, yet should be.

Dimitri John Ledkov (xnox) wrote :

http://paste.ubuntu.com/1282746/

We should probably change the: disable-lock-screen key as well.

no longer affects: indicator-session (Ubuntu)
summary: - nothing happens after click the "start screen saver" button on the top-
- right corner of the screen during the installation progress
+ "start screen saver" button on the top-right corner of the screen during
+ the installation progress should be removed
summary: - "start screen saver" button on the top-right corner of the screen during
- the installation progress should be removed
+ "start screen saver" does nothing if lock-screen is locked down
description: updated
Changed in ubiquity (Ubuntu):
status: Triaged → Invalid
Changed in indicator-session (Ubuntu):
status: New → Confirmed
Charles Kerr (charlesk) on 2012-10-18
Changed in indicator-session:
status: New → Confirmed
assignee: nobody → Charles Kerr (charlesk)
Charles Kerr (charlesk) on 2012-10-24
Changed in indicator-session:
status: Confirmed → Fix Committed
milestone: none → 13.04.0
Omer Akram (om26er) on 2012-11-13
no longer affects: ubiquity (Ubuntu)
Changed in indicator-session:
importance: Undecided → Low
Changed in indicator-session (Ubuntu):
importance: Undecided → Low
status: Confirmed → Fix Committed
assignee: nobody → Charles Kerr (charlesk)
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-session - 12.10.5daily13.01.25-0ubuntu1

---------------
indicator-session (12.10.5daily13.01.25-0ubuntu1) raring; urgency=low

  [ Mathieu Trudel-Lapierre ]
  * debian/rules:
    - Add DPKG_GENSYMBOLS_CHECK_LEVEL=4.
    - Use the dh sequencer with autoreconf.
    - Override dh_autoreconf to run autogen.sh.
  * debian/control:
    - Add dh-autoreconf to Build-Depends.
    - Add gnome-common to Build-Depends.
    - Add libgtest-dev, libdbustest1-dev, dbus-test-runner to Build-Depends.
    - Adjust style to be consistent and use trailing commas in lists.
    - Update Vcs-Bzr and Vcs-Browser fields and add notice for uploaders.
    - Remove libpackagekit-glib2-dev and libpolkit-gobject-1-dev
      from Build-Depends. (LP: #1085100)
    - Remove packagekit from Recommends
    - Add XS-Testsuite: autopkgtest.
    - Add libxorg-gtest-dev to Build-Depends.
  * debian/tests:
    - Add start-service test; verify that the indicator can be started from
      DBUS.
    - Add debian/tests/control.
  * Automatic snapshot from revision 373 (bootstrap):
    - Check to make sure that create_display_proxy() succeeded before using the
      proxy pointer that gets returned. (LP: #1078119)
    - Don't lock the session when clicking on the current user. (LP: #1024392)
    - Hide 'start screen saver' action, if it's locked down. (LP: #1066825)
    - Use the same icons in the greeter as in the user session (LP: #1049244)
    - Test upower_proxy for NULL before using it. (LP: #1061844)
  * Move TestCanStartService into a separate make target "localcheck"; so that
    the other tests can successfully run, and this one be used as an
    integration check.
  * debian/control: temporarily comment out the XS-Testsuite entry to disable
    autopkgtest until the tests really work properly.

  [ Sebastien Bacher ]
  * debian/control:
    - clean deprecated Build-Depends on gconf and dbusglib

  [ Didier Roche ]
  * run tests not under fakeroot for dbus tests (which don't seem to be able to
    run easily under dbus-test-runner)

  [ Charles Kerr ]
  * Tests hanging (LP: #1074065)
  * User-switching item is blank if there's no display name (LP:
    #1076271)
  * unnecessary dependencies on polkit and packagekit (LP: #1085100)

  [ Robert Ancell ]
  * Replace dbus-glib with GDBus (LP: #1084756)

  [ Automatic PS uploader ]
  * Automatic snapshot from revision 382
 -- Automatic PS uploader <email address hidden> Fri, 25 Jan 2013 02:01:17 +0000

Changed in indicator-session (Ubuntu):
status: Fix Committed → Fix Released
Charles Kerr (charlesk) on 2014-03-13
Changed in indicator-session:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers