You're right about that. EmptyPanel is better, and it should probably be final.
Eelco On 5/5/07, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
will that really make things better? if you misname the markup file you will now get component not found rather then markup not found. -igor On 5/5/07, Martijn Dashorst <[EMAIL PROTECTED]> wrote: > > How about giving Panel.java an accompanying Panel.html file? That > would obviate the need of adding EmptyPanel, and anyone subclassing > Panel would override it by supplying their own markup. > > Martijn > > On 5/5/07, Martijn Dashorst <[EMAIL PROTECTED]> wrote: > > +1, another class I can remove from our project :) > > > > On 5/5/07, Jonathan Locke <[EMAIL PROTECTED]> wrote: > > > > > > > > > so i propose a vote to add this. i'm +1. > > > > > > > > > Eelco Hillenius wrote: > > > > > > > > Yeah. I actually think I proposed something like this, or at least > > > > +1-ed one at more than a year ago. You'll have to dig in the > archives > > > > to find out why other people didn't agree back then, but I still > think > > > > it's much easier for users to discover something just because it has > > > > an obvious name, and not to worry about that one extra class. > > > > > > > > Eelco > > > > > > > > > > > > On 5/4/07, Jonathan Locke <[EMAIL PROTECTED]> wrote: > > > >> > > > >> In my work project, I've added and frequently use an EmptyPanel > which is > > > >> simply a panel with empty markup. Although you can do the same > thing > > > >> with > > > >> WebMarkupContainer or implement this yourself, it does seem like a > nice > > > >> conceptual addition, particularly for newbies who might not think > "hmm... > > > >> i > > > >> want an empty panel instead of this fancy panel if the user isn't > logged > > > >> in... oh WebMarkupContainer looks about right..." ;-) Anyway, I > thought > > > >> I'd > > > >> mention it in case others feel this would be a nice addition to > core. > > > >> -- > > > >> View this message in context: > > > >> http://www.nabble.com/EmptyPanel--tf3692546.html#a10324363 > > > >> Sent from the Wicket - Dev mailing list archive at Nabble.com. > > > >> > > > >> > > > > > > > > > > > > > > -- > > > View this message in context: > http://www.nabble.com/EmptyPanel--tf3692546.html#a10333463 > > > Sent from the Wicket - Dev mailing list archive at Nabble.com. > > > > > > > > > > > > -- > > Learn Wicket at ApacheCon Europe: http://apachecon.com > > Join the wicket community at irc.freenode.net: ##wicket > > Wicket 1.2.6 contains a very important fix. Download Wicket now! > > http://wicketframework.org > > > > > -- > Learn Wicket at ApacheCon Europe: http://apachecon.com > Join the wicket community at irc.freenode.net: ##wicket > Wicket 1.2.6 contains a very important fix. Download Wicket now! > http://wicketframework.org >
