Adium

Ticket #112 (closed defect: fixed)

Opened 7 years ago

Last modified 3 years ago

Authorize dialog has incorrect button as default.

Reported by: huw.rowlands@gmail.com Owned by: evands
Milestone: Adium X 1.0 Component: Adium Core
Version: 0.81 Severity: normal
Keywords: authorize default button Cc:
Patch Status:

Description

When a buddy adds you to their list a dialog comes up informing you that they have added you. You then get the option to 'accept' or 'refuse' them. For some reason, 'refuse' is the default button. This is potentially misleading and inconsistent.

Apple's HIG has certain specifications concerning such a thing:

"Button names should correspond to the action the user performs when pressing the button, for example, Erase, Save, or Delete. The rightmost button in the dialog, the action button, is the button that confirms the alert message text. The action button is usually, but not always, the default button."

Currently the question in the dialog box is "Do you want to accept them?" and the default answer is "no." The default button should be the affirmative, but here it is not.

"Don't use a default button if the most likely action is dangerous—for example, if it causes a loss of user data. When there is no default button, pressing Return or Enter has no effect; the user must explicitly click a button. This guideline protects users from accidentally damaging their work by pressing Return or Enter. You can consider using a safe default button, such as Cancel."

'Refusing' a request currently no record of the request having been made is kept and there is no way of undoing the 'refuse' if it was unintentional. One has to go into the official client to undo the damage. On the other hand, if the 'accept' button was the default then the use is quite able to undo the damage by simply blocking the user in question (works, afaik).

Change History

comment:1 Changed 7 years ago by catfish_man

  • Milestone set to Adium X 0.82

comment:2 Changed 7 years ago by evands

  • Owner changed from anybody to evands
  • Status changed from new to assigned
  • Milestone changed from Adium X 0.82 to Adium X 0.90

comment:3 Changed 7 years ago by evands

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed in [12071]

comment:4 Changed 7 years ago by catfish_man

  • Milestone changed from Adium X 0.90 (Old) to Adium X 0.90

comment:5 Changed 3 years ago by zacw

  • Component changed from Core Adium to Adium Core
Note: See TracTickets for help on using tickets.