Button labels phrasing

Bug #140472 reported by Wouter Stomp
8
Affects Status Importance Assigned to Milestone
apturl (Ubuntu)
Fix Released
Wishlist
Siegfried Gevatter

Bug Description

Binary package hint: apturl

According to the HIG guidelines, button labels should have imperative verbs. So please change the buttons in the apturl dialog to something like “Don’t install” & “Install” instead of Yes/No buttons

Related branches

Murat Gunes (mgunes)
Changed in apturl:
importance: Undecided → Wishlist
Revision history for this message
Bruce Cowan (bruce89-deactivatedaccount) wrote :

Here's a patch, not sure if it works (apturl no longer works in Hardy thanks to XULRunner 1.9).

Revision history for this message
Michael Vogt (mvo) wrote :

Thanks for your bugreport and your patch.

We are in UI freeze currently so would like to postpone this patch for hardy+1.

Changed in apturl:
milestone: none → later
status: New → Confirmed
Revision history for this message
Siegfried Gevatter (rainct) wrote :

I've applied Bruce's patch to my branch (revision 50).

Changed in apturl:
status: Confirmed → Triaged
assignee: nobody → rainct
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apturl - 0.2.7ubuntu1

---------------
apturl (0.2.7ubuntu1) intrepid; urgency=low

  [ Siegfried-Angel Gevatter Pujals ]
  * apturl:
     - Show a different error message if the requested package is virtual; this
       is the first step to fix Launchpad bug #230760.
     - Show an error dialog if the /etc/apt/sources.list is invalid instead of
       just crashing (LP: #206640).
     - Catch KeyboardInterrupt's so that they don't show a traceback.
     - Add a new exit code (RESULT_BADARGS, 4) for when the problem is with
       the arguments with which apturl was called.
     - Only ask once when enabling new components, and only list those
       which aren't already enabled.
     - Fix a crash which happened after enabling the requested component(s).
     - Don't show an error message if the user didn't want to enable a
       component; just silently skip the package.
     - Improve the description:
        + Don't show the summary twice.
        + Remove single line breaks, like packages.ubuntu.com does.
        + Parse the Homepage field out of the description.
     - Do not ask if you want to keep a software channel if it wasn't added.
     - Use a constant to determine wheter new software channels can be added
       or not (disabled by default, but that's better than having the code
       commented out).
  * AptUrl/Helpers.py:
     - Add it.
  * data/apturl.glade:
     - Add a missing "can_default" property to fix an assertion error.
     - Change the labels of the buttons on the installation confirmation dialog
       so that they conform to the HIG. Thanks to Bruce Cowan (LP: #140472).
     - Make the description field bigger and more visually appealing.
  * AptUrl/Parser.py:
     - Raise an Exception if one of the given package names contains characters
       which are not allowed by the Debian Policy.
     - Raise an Exception if one of the ?... values contains a space.
  * apturl.8:
     - Document all possible exit codes.
     - Add a new example, and improve the comment of an existing one.
     - Change the URL in "Bugs" from /apturl to /ubuntu/+source/apturl, as
       that's where most bugs are.
  * debian/control:
     - Add Vcs-Browser field.
     - Write all dependendency and build dependencies each on a line.
     - Add python-gobject to the build dependencies (needed for the timer).
  * debian/rules:
     - Fix the clean rule so that it removes build/ and AptUrl/Version.py.
     - Move the stuff from binary-arch to binary-indep, and remove the
       binary-indep target to please lintian.
  * debian/copyright:
     - Update copyright years and author names.

  [ Артём Попов ]
  * data/apturl.protocol, setup.py:
     - Register apturl in KDE as a protocol (LP: #227622).

 -- Siegfried-Angel Gevatter Pujals <email address hidden> Mon, 01 Sep 2008 16:44:35 +0200

Changed in apturl:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Patches

Remote bug watches

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