Re: [evolution-patches] Re: Various libebook patches



On Wed, 2004-01-07 at 13:55, Chris Toshok wrote:
> On Wed, 2004-01-07 at 07:01, Dan Winship wrote:
> > > > If we're going to have default folders, we need to do them for real like
> > > > in 1.4. If we're not, then the user always needs to be able to choose
> > > > what folder to use, and the code always needs to work with ESources, and
> > > > there shouldn't be a "get local addressbook" call, because it will just
> > > > encourage people to write broken code.
> > > 
> > > I had totally forgotten about the "Default Folders" option in
> > > Evolution...
> > > 
> > > Keep e_book_get_default_addressbook() and it should return what the user
> > > has set as the Default in Evo. Makes perfect sense!
> > 
> > The problem is that the current plan is to not have the default folders
> > settings in 2.0.
> 
> Yeah, not having default folders is really the only reason to have the
> local addressook call.  It's the only addressbook we can be sure should
> exist (and that the user might want to use to store stuff) and a call
> like that goes a long way to making things easier for people writing
> ebook code, especially where you don't really want to make the user make
> a choice of all their possible contact folders (non-gui apps).

But its mostly just a matter of a little code to do things right.  The
addressbook conduit uses the local call but thats just a place holder,
it really needs to allow the user to select a source (possibly multiple
so that one source maps to one category on the palm).  Of the other two
calls in the calendar, one is useless and the other is borked since its
used to look up free/busy information, but the information can be in any
number of addressbook backends.

> Implementing a default folder setting is trivial with the new ESource
> stuff (just add a defaultfolder="true" property to that source.)  We
> *could* make the default folder stuff actually work and just not use it
> from evolution.  Add an e_book_set_default_addressbook call as well,
> make it default to "On This Computer/Personal"...

What we really need to do is make stuff like the ESourceSelector and
ESourceOptionMenu available for people to use in their apps, then a lot
of the selection of sources becomes easy.

-JP
-- 
JP Rosevear <jpr ximian com>
Ximian, Inc.




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]