On Mon, Nov 29, 2010 at 9:35 AM, Sven Meier <s...@meiers.net> wrote:
> 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? ;)

yes

-igor

>
> 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/
>>
>
>
>

Reply via email to