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
