Low disk space warning "Cancel" and "OK" buttons do the same thing

Bug #337441 reported by Matthew Paul Thomas on 2009-03-03
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNOME Settings Daemon
Fix Released
Undecided
Unassigned
gnome-settings-daemon (Ubuntu)
Low
Chris Coulson

Bug Description

Binary package hint: gnome-settings-daemon

In Gnome 2.26, gnome-settings-daemon includes a new warning about low disk space <http://bugzilla.gnome.org/show_bug.cgi?id=557647>. This is implemented using a non-expiring notification bubble, which Notify OSD renders as a fallback alert box with "Cancel" and "OK" buttons. The buttons do exactly the same thing, i.e. close the alert box.

Instead, the warning should use a custom alert box that opens unfocused. Mini-spec: <https://wiki.ubuntu.com/NotifyOSD#gnome-settings-daemon>

Sebastien Bacher (seb128) wrote :

Thank you for your bug report

Changed in gnome-settings-daemon:
assignee: nobody → desktop-bugs
importance: Undecided → Low
milestone: none → ubuntu-9.04-beta
status: New → Confirmed
Changed in gnome-settings-daemon (Ubuntu):
milestone: ubuntu-9.04-beta → ubuntu-9.04
Changed in gnome-settings-daemon (Ubuntu):
assignee: Ubuntu Desktop Bugs (desktop-bugs) → Chris Coulson (chrisccoulson)
status: Confirmed → In Progress
Chris Coulson (chrisccoulson) wrote :

This is now fixed upstream with this commit, and will be in the next version: http://git.gnome.org/cgit/gnome-settings-daemon/commit/?id=8f1e770acb92682d1a5a4b7dd16049155cad2902

Changed in gnome-settings-daemon (Ubuntu):
status: In Progress → Fix Committed
Changed in gnome-settings-daemon:
status: New → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-settings-daemon - 2.27.4-0ubuntu1

---------------
gnome-settings-daemon (2.27.4-0ubuntu1) karmic; urgency=low

  * New upstream version (LP: #401510):
    - Remove screensaver plugin, it's autostarted now
      (Matthias Clasen)
    - Don't take too long in RANDR D-Bus method implementation
      (Federico Mena Quintero)
    - Add support for Synaptics touchpads (Matthias Clasen)
    - Don't spawn more than one syncdaemon (Matthias Clasen)
    - Depend on gnome-desktop >= 2.26.3 (Rodrigo Moya)
    - Update gnome-volume-control code from master (Bastien Nocera)
    - Fix order of arguments to strstr (Federico Mena Quintero)
    - Depend on libxklavier 4.0 (Sergey V. Udaltsov)
    - Remove libglade dependency from media-keys and keyboard plugins
      (Felix Riemann)
  * debian/control{.in}:
    - Bump libgnome-desktop-dev build-dependency.
    - Migrate libxklavier12-dev build-dep to libxklavier-dev and bump
      version requirement.
    - Build-depend on libpulse-dev.
  * Refreshed patches to apply cleanly again.
  * Dropped 07_use_gstreamer_volume.patch to use the Pulseaudio
    volume control rather than the old GstMixer interface.
  * Dropped 08_extra_touchpad_options.patch - fixed upstream now.
  * debian/gnome-settings-daemon.install:
    - Updated to correctly install *.ui files and control-center
      xml file.
  * debian/patches/50_ldsm_notification.patch:
    - Upstream change to implement the new low disk space warning
      (Fixes LP: #337441, LP: #390504 and LP: #396864).
  * debian/patches/60_ldsm_notification_fixup.patch:
    - Fix some bugs in the upstream implementation of the last patch,
      eg, I made the assumption that all mounts with a device node
      not beginning with /dev could be ignored as a way of filtering out
      uninteresting volumes. This is not so, as the GUnixMountEntry for
      the root filesystem can have "rootfs" as the device node, and
      we don't want to filter the root filesystem out.
    - Make sure that no warning is shown for volumes with more than 2GB
      free space by default.

 -- Chris Coulson <email address hidden> Thu, 23 Jul 2009 21:34:51 +0100

Changed in gnome-settings-daemon (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers