Re: [evolution-patches] big setting patch

[Jeffrey Stedfast <fejj ximian com>]

This patch seems "mostly okay" except for listening to changes for all
/apps/evolution/mail/display/* setting changes. This bothers me only
because there are settings under that hierarchy that should not cause a
redisplay of the message - these settings would include:

/apps/evolution/mail/display/mark_seen
/apps/evolution/mail/display/mark_seen_timeout
/apps/evolution/mail/display/show_deleted
/apps/evolution/mail/display/show_preview
/apps/evolution/mail/display/paned_size
/apps/evolution/mail/display/thread_list
/apps/evolution/mail/display/thread_subject

I think the best approach for now would just be to listen to the
individual keys that *should* cause a redisplay.

However, I think the ideal solution might be to rearrange the gconf keys
in such a way as to separate out the keys listed above to not be under
display/ and to instead be grouped somewhere else. For example, we might
want to move a number of those keys into, say:

/apps/evolution/mail/message_list/

mark_seen, mark_seen_timeout, show_deleted, thread_list and
thread_subject would all be good candidates for this.

Not sure where the best place for paned_size or show_preview might be,
maybe the toplevel? (/apps/evolution/mail/)

Jeff

On Tue, 2003-05-20 at 12:22, Larry Ewing wrote:
> This hooks up the settings that were not hooked up and add the
> appropriate schema entries.
> 
> --Larry
-- 
Jeffrey Stedfast
Evolution Hacker - Ximian, Inc.
fejj ximian com  - www.ximian.com