On Thu, Feb 18, 2016 at 9:15 AM, Martin Grigorov <[email protected]>
wrote:

> On Thu, Feb 18, 2016 at 10:12 AM, Urbani, Edmund <
> [email protected]
> > wrote:
>
> > Hello,
> >
> > I have at times used factories and then found that they made some simply
> > things more complicated. Eg. I could no longer simply override
> isEnabled()
> > or isVisible() methods to suit the page I was
> >
>
> Overriding isVisible/isEnabled() is a bad idea anyway!
> Use onConfigure() + setVisible/setEnabled instead.
>


Oh dear, I override isVisible/isEnabled quite a lot.
Why is it considered a bad idea?

Reply via email to