Re: [evolution-patches] Patch for #34354

[Rodney Dawes <dobey novell com>]

Part of the patch is irrelevant to the fix and just makes whitespace
changes. Does the change make the dialog modal to all Evolution windows,
or only the one that it was opened from? It certainly would make sense
to allow other Evolution windows to switch between components and
folders I think. If the changes doesn't block other windows, I think it
is OK to put in HEAD, and possibly even the branch. It's a minor bug
fix, and not a major UI change that would break documentation. I am ok
with it only going in HEAD as well.

-- dobey


On Thu, 2005-03-24 at 10:18 -0500, JP Rosevear wrote:
> On Sat, 2005-03-26 at 23:01 +0800, Mengjie Yu wrote:
> > hi, Rodney
> > 
> > Here is the patch for #34354, and it is about UI, so it need your
> > review.
> > 
> > the related URL is:
> > http://bugzilla.ximian.com/show_bug.cgi?id=34354
> > 
> > Will you please help me to review it?
> 
> I think this should only go to HEAD if accepted.
> 
> -JP