Re: [PATCH] Add a way to set the theme for an HippoCanvasBox

On Wed, 2008-06-04 at 15:07 +0200, Marco Pesenti Gritti wrote:
> Owen,
> is this what you had in mind?
> Can you see any problem with the implemenation? (I'm not very familiar
> with hippo style inheritance/propagation).

Yeah, that's what I meant. The implementation looks good - the only
potential issue I see - and it looks like the problem is already there
with the current code - is that since the context is stored in the style
(and also affects cached properties) the style needs to be cleared when
the context changes

- Owen

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