--- On Thu, 15/9/11, Colomban Wendling lists.ban@herbesfolles.org wrote:
How about adding a PREF_CANCEL action to prefs.c only
that will call a
stash_group_cancel() to reset the values? The other
I don't think that's a good design, the stash group settings should not actually be changed until ok/apply.
Perhaps you meant stash_tree_cancel(), but see below.
option is to clear
the store, but that means recreating the various list
[a visibly slow
process] each time Edit -> Preferences is invoked,
while traversing the
model is fast.
What about traversing the model on reshow to reset it to the default values?
Yes, that would be best. (This would be similar to how all stash prefs work with stash_group_display - the widgets are updated with the current settings).
(though honestly building the view don't seem any slow to me)
I didn't notice any slowness, but it would be interesting to know why if it is.
Nick