so, my only argument against adding this without addressing the inheritance
issue is that once we do this, it will be very difficult to change. i.e. we
won't be able to change the default behavior. It will require us adding an
extra flag on the widget, indicating whether or not to inherit the paren't
font.

I'm just saying it's worth noodling on for a bit.


On Sun, Feb 10, 2013 at 12:39 PM, Pietro Berkes <notifications@github.com>wrote:

> In enaml/qt/qt_widget.py:
>
> > @@ -265,7 +266,8 @@ def set_font(self, font):
> >              The font for the widget as a CSS font string.
> >
> >          """
> > -        pass
> > +
> > +        self.widget().setFont(QtGlobalFontCache[font])
>
> I agree about testing font != '' , I'll fix it.
> As for the automatic inheritance, I'd rather leave it for another PR. This
> one restores the basic support for an attribute that worked on the 'legacy'
> branch but does not work on the async branch.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/enthought/enaml/pull/256/files#r2955601>..
>
>