If you worry about memory consumption, the visitor pattern is
the way to go. Thing is, I only want the link resolution to occur the
first time the Page is rendered (for performance reasons) so the
Javadoc should explicitly warn that this is what happens (i.e. the
method is guaranteed to be called at least once per Page lifetime to
resolve autolinks).
One final thought: autolinks shouldn't be handled by the Page
because by their very nature they're meant to be automatic. If you can
think up of a usecase where a Page would want to actually be making the
decision instead of ... Container.java or something ... please let me
know.
Gili
On Wed, 5 Jan 2005 10:07:11 +0100, Juergen Donnerstag wrote:
>Yes and I have until tomorrow no CVS acces (only email). And I have
>not yet found the perfect solution to implement it. I've got
>implementation which creates a ComponentWicketTag for each tag
>containing a href attribute. But I don't like that approach because
>we'll finally end up with one ComponentTag for almost every XML tag.
>But it is working. The other idea I have: allow register a listener
>with MarkupParser each gets called with each tag, e.g. Page could
>register one (autolinks are Page relative and let the listener resolve
>the url. Con.: we'll end up with two different means of interpreting
>markup: a) MarkupElement and b) Listeners.
>
>Not sure which approach is the best. Any other ideas?
>
>Juergen
>
>
>On Tue, 04 Jan 2005 18:58:03 +0100, Eelco Hillenius
><[EMAIL PROTECTED]> wrote:
>> Juergen is still working on that one.
>>
>> Eelco
>>
>> Gili wrote:
>>
>> >Hi,
>> >
>> > I'm taking a look at the new example code that's supposed to
>> >use the new namespace and autolink approaches. It seems to still use:
>> >
>> ><h2><a id="wicket-[autolink]" href="DisplaytagIndex.html">Example</a>
>> >> Standard, smart linking of column data</h2>
>> >
>> > in ExampleAutolink.html. So first thing is, we should replace
>> >this with the new approach. Secondly, what is the new syntax? Is this
>> >correct?
>> >
>> ><h2><wicket:link autolink="true"
>> >href="DisplaytagIndex.html">Example</wicket:link> > Standard, smart
>> >linking of column data</h2>
>> >
>> > (I forget...)
>> >
>> >Gili
>> >
>> >
>> >
>> >-------------------------------------------------------
>> >The SF.Net email is sponsored by: Beat the post-holiday blues
>> >Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
>> >It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
>> >_______________________________________________
>> >Wicket-user mailing list
>> >[email protected]
>> >https://lists.sourceforge.net/lists/listinfo/wicket-user
>> >
>> >
>>
>> -------------------------------------------------------
>> The SF.Net email is sponsored by: Beat the post-holiday blues
>> Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
>> It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
>> _______________________________________________
>> Wicket-user mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/wicket-user
>>
>
>
>-------------------------------------------------------
>The SF.Net email is sponsored by: Beat the post-holiday blues
>Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
>It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
>_______________________________________________
>Wicket-user mailing list
>[email protected]
>https://lists.sourceforge.net/lists/listinfo/wicket-user
>
-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
_______________________________________________
Wicket-user mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-user
