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

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

Bug Description

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

description: updated
tags: added: rls-q-incoming
Revision history for this message
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)
Revision history for this message
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?

Revision history for this message
Victor Zhou (victor.zhou) wrote :

to Dmitrijs Ledkovs,
the attachment is the screen shot.

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Thanks.... it ought to be removed.

affects: casper (Ubuntu) → ubiquity (Ubuntu)
Changed in ubiquity (Ubuntu):
status: New → Triaged
Revision history for this message
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.

Revision history for this message
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)
Changed in indicator-session:
status: New → Confirmed
assignee: nobody → Charles Kerr (charlesk)
Charles Kerr (charlesk)
Changed in indicator-session:
status: Confirmed → Fix Committed
milestone: none → 13.04.0
Omer Akram (om26er)
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)
Revision history for this message
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)
Changed in indicator-session:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.