Hi Douglas, WICKET-3171 describes a problematic case, where visibility of a component changes while its form is being processed. In our projects we're overriding isVisible() where appropriate and never encountered a similar problem.
I'd say WICKET-3171 is the rare 5% usecase. What's next, is overriding isEnabled() going to be declared evil too? ;) Sven On Mon, 2010-11-29 at 11:22 -0600, Douglas Ferguson wrote: > Can you explain why? We have done this all over the place. > > D/ > > On Nov 29, 2010, at 10:00 AM, Martin Grigorov wrote: > > > The recommended way since a few 1.4 releases is to override onConfigure() > > and call setVisible(true|false) depending on your conditions. > > > > On Mon, Nov 29, 2010 at 4:49 PM, Douglas Ferguson < > > doug...@douglasferguson.us> wrote: > > > >> Igor posted a comment to this bug saying that overriding isVisible() is > >> "evil" > >> > >> https://issues.apache.org/jira/browse/WICKET-3171 > >> > >> I was surprised by this and am curious to hear more. > >> > >> D/ >