Adium

Ticket #6190 (closed enhancement: fixed)

Opened 2 years ago

Last modified 1 year ago

Move Accounts to the left of the prefs window

Reported by: cbarrett Assigned to: cbarrett
Priority: normal Milestone: Adium X 1.1
Component: Adium UI Version:
Severity: normal Keywords:
Cc: evan@adiumx.com Patch:
Pending:

Description

From devl@:

Maybe accounts should be on the left, with a vertical separator between it and General. Accounts really isn't a preference pane, but we're putting it in our preferences window for usability reasons.

Further discussion should be held here.

Change History

01/27/2007 01:29:04 AM changed by boredzo

Seconded.

(Note: I'm a developer. Just because I just said “seconded” is not an invitation for every user who reads this to add their own vote. :)

02/06/2007 06:52:40 PM changed by tick

  • milestone changed from Adium X 1.1 to Needs dev review.

Discussions like this have to take place on the dev milestone first.

(follow-up: ↓ 4 ) 02/06/2007 11:06:25 PM changed by cbarrett

  • milestone changed from Needs dev review to Adium X 1.1.

Not when they have been discussed on devl@ already.

(in reply to: ↑ 3 ) 02/07/2007 12:36:34 AM changed by tick

Replying to cbarrett:

Not when they have been discussed on devl@ already.

So put them into this ticket.

(follow-up: ↓ 7 ) 02/07/2007 08:33:37 AM changed by cbarrett

The relevant discussion is archived on devl@ here: http://adiumx.com/pipermail/adium-devl_adiumx.com/2007-January/001699.html (search for vertical).

Please read devl@. If not everyone who is involved in the project reads it regularly, then why is it even there (alternately, if they don't have time, maybe they should cut back their involvement in the project accordingly).

(follow-up: ↓ 8 ) 02/14/2007 09:34:20 AM changed by cbarrett

  • cc set to evan@adiumx.com.

Did some investigating on this.

AIPreferenceWindowController.m and PreferenceWindow.nib are the class and nib.

I'm not exactly sure how the actual tab view is being generated, with the icons and what not. Is that the NSTabView doing that? If that is in a different file, where is it?

Tagging Evan with tracmail so he can answer at his leisure

(in reply to: ↑ 5 ) 03/04/2007 12:50:34 AM changed by tick

Replying to cbarrett:

The relevant discussion is archived on devl@ here: http://adiumx.com/pipermail/adium-devl_adiumx.com/2007-January/001699.html (search for vertical). Please read devl@. If not everyone who is involved in the project reads it regularly, then why is it even there (alternately, if they don't have time, maybe they should cut back their involvement in the project accordingly).

1) I wouldn't call that "discussed". You mentioned it, Peter thought it was a good idea, nobody else chimed in.

2) People other than those subscribed to devl read trac tickets. Some of them may potentially become developers on the project. Minus the context of the "discussion" on devl, there's nothing here that shows how this was decided.

Replying to cbarrett:

Please read devl@. If not everyone who is involved in the project reads it regularly, then why is it even there (alternately, if they don't have time, maybe they should cut back their involvement in the project accordingly).

Trac is not an appropriate forum for comments like these.

(in reply to: ↑ 6 ) 03/05/2007 05:02:24 PM changed by evands

Replying to cbarrett:

Did some investigating on this. AIPreferenceWindowController.m and PreferenceWindow.nib are the class and nib. I'm not exactly sure how the actual tab view is being generated, with the icons and what not. Is that the NSTabView doing that? If that is in a different file, where is it?

Sorry for the delay; forgot to reply after I marked this read.

AIPreferenceWindowController specifies the ordering via an array... but the nib needs to be updated to match, I believe. Classic case of 'overcomplicated but works... so touch it lightly or rewrite it entirely :P'

04/07/2007 09:27:28 PM changed by evands

  • status changed from new to closed.
  • resolution set to fixed.

(In [19325]) Reworked how the preferences window is generated and displayed.

  • We now use Matt Gemmell's SS_PrefsController (http://mattgemmell.com/ - updates to readme, etc. to follow) to dynamically generate the preferences window. There are some Adium-specific modifications to it, a fix for initial-selection, and the ability to include separators.
    • The preferences window can now allow resizing on a per-view basis if we want to do that. Refs #4380
    • The window will size both horizontally and vertically as necessary to display preferences. This should fix #6248.
    • AIAdvancedPreferencePane is now the superclass for advanced preferences.
    • The accounts pane is now first and separated from the rest by a separator. Fixes #6190.
  • Preferences nibs now have full control over their display. Any border between the view and the edge of the window is specified by that nib. This makes further modifications to the prefs much less finicky. Refs #1882.
  • The Advanced preferences UI now uses a source list. In the process, a minor drawing bug was corrected. Fixes #4708
  • Message history preferences are now in General. The advanced preferences for it have been removed from that pane. The behavior can be customized via the Customize button next to the checkbox. Fixes #6682.
  • General now utilizes bindings for most of its preferences.