User login controller "patch"

[Evan Schoenberg]

On Jun 5, 2009, at 9:47 AM, Chris Forsythe wrote:

> Is this a patch that they could make into a plugin to be put onto  
> the xtras site?

I wouldn't suggest that route to our interested patch writer.  Here's  
someone who is working on a new feature that they feel the community  
at large will benefit from; I don't think they should be pushed toward  
doing it Wrong and being able to publish it to a comparatively small  
audience versus doing it Right and having it included in Adium itself.

I fully agree that the login controller stuff is a hack and shouldn't  
be encouraged.  In fact, once we have a better alternative in place  
for end users, we should #ifdef DEBUG_BUILD the whole conditional code  
such that releases never show the option to change profiles.

What if the Accounts pane had a popup menu at the top, like the  
Location: dropdown above the source list in the Network system  
preferences? The bottom item of the menu would be the usual "Edit  
Profiles" (or whatever word we use to describe sets of accounts).  The  
Edit Profiles pane, in turn, could link a given profile to a given  
Location ("[ ] Automatically activate when in location <-LOCATION  
CHOICES->").

Cheers,
Evan

>
>
> Chris
>
> On Jun 4, 2009, at 1:33 PM, Zachary West wrote:
>
>> Hi,
>>
>> Someone in #adium-devl has been working on something that I've been
>> hard-lining against, so I figured I'd bring it here for discussion.
>>
>> His idea is to add network awareness to the login controller (option
>> when starting Adium), so a user profile is tied to a location.
>>
>> As a totally not-user-facing and not-supported-in-the-least-bit
>> feature of Adium, not to mention one that lacks a way to switch users
>> without restarting Adium, or any number of problems (total preference
>> loss, total log loss, etc, on switching) I've been vehemently against
>> accepting any patch that uses the login controller.
>>
>> I wanted to gauge devel's level of agreement with me. I don't think
>> any patch which relies on the login controller should be accepted,  
>> and
>> this is better served as a general "this account in this network",
>> etc, part of an advanced preference (if at all).
>>
>> Zac
>>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://adium.im/pipermail/devel_adium.im/attachments/20090605/811f85ff/attachment-0002.html>