I don't like the re-parenting idea for this use case very well because
we have two hierarchies: 1) components and b) markup. Currently
getParent() is clearly the component hierarchy and IMO we shouldn't
change it. getMarkupFragment() needs to be intelligent enough to find
the markup in the markup hierarchy. If that means we not only search
in the container but also in all transparent childs, than I think it
is worth a trial.

Juergen

On 9/12/06, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
> i dont think this is a case of a transparent resolver - even though they
> couldve been used previously. what we need is to do this to properly support
> it
>
> make wicket:body a webmarkupcontainer
> when a child is added to a border the border needs to check where the markup
> is
> if the markup is in the border add it to the border
> if the markup is inside wicket:child add it to the created wicket:child
>
> this will maintain a consistent hierarchy
>
> what we need is something i talked about before...
>
> IParentResolver { MarkupContainer getParent(); } - that allows a component
> to redirect its children to something else (usually something inside it).
> that way a border component can implement this interface and redirect
> children to its internal wicket:child markup container. implementations of
> this will need to be careful because they will be working on a
> non-initialized class since the check for IParentResolver will be in the
> Component constructor.
>
> johan and i had discussed it previously, but that discussion has stalled
> until one of us had time to seriously look into this.
>
>
> -Igor
>
> On 9/12/06, Martijn Dashorst <[EMAIL PROTECTED]> wrote:
> > It is a case that could be solved previously by using transparent
> > resolvers, which were needed for borders. So if we need it for this
> > usecase, then we could solve the border problem along the way ;-)
> >
> > Martijn
> >
> > On 9/12/06, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
> > > i dont think thats exactly the same as what we originally talked about.
> but
> > > i do think we need to support this usecase as well, i hit it a while ago
> > > with a <form> instead of <div>
> > >
> > > -Igor
> > >
> > >
> > >
> > >  On 9/12/06, Martijn Dashorst <[EMAIL PROTECTED]> wrote:
> > > > Well, I can see a usecase for this, including markup inheritance:
> > > >
> > > > <div wicket:id="modifyattrsonserverOrDoAjaxyStuff">
> > > > <wicket:child />
> > > > </div>
> > > >
> > > > though the div might not be a border per se (WebMarkupContainer
> > > > springs to mind)... I think we need to support this case.
> > > >
> > > > Martijn
> > > >
> > > > On 9/12/06, Juergen Donnerstag < [EMAIL PROTECTED]> wrote:
> > > > > It might work, but in opinon it is feature rarely used especially as
> > > > > markup inheritance is avaiable as well. IMO it is too much magic
> > > > > besides that it would allow to remove some "just because of bordered
> > > > > pages" code in several places. IMO the feature is not worth the
> extra
> > > > > code.
> > > > >
> > > > > Juergen
> > > > >
> > > > > On 9/12/06, Martijn Dashorst <[EMAIL PROTECTED]> wrote:
> > > > > > I think we could limit the transparent resolving only to
> components
> > > > > > that are already definied in the flow. I don't think this is a too
> > > > > > severe restriction, but will keep the door open for current
> usecases.
> > > > > >
> > > > > > Martijn
> > > > > >
> > > > > > On 9/12/06, Martijn Dashorst < [EMAIL PROTECTED]> wrote:
> > > > > > > Hmm,
> > > > > > >
> > > > > > > just a thought, but not completely thought through though...
> > > > > > >
> > > > > > > The auto border could be set as an autoparent, in the base
> > > > > > > constructor. This would give Wicket the means to use the border
> as
> > > an
> > > > > > > auto look up thing.
> > > > > > >
> > > > > > > The idea was that in your base class you define the border as
> auto.
> > > As
> > > > > > > the super constructor is always called first, you can be sure
> (as a
> > > > > > > user) that your tranparent resolver is already in place when the
> new
> > > > > > > components are added.
> > > > > > >
> > > > > > > As for the parent path... when a component is resolved to a
> > > > > > > transparent parent, why not set the parent automagically to the
> > > border
> > > > > > > ourselves?
> > > > > > >
> > > > > > > The search algorithm could be:
> > > > > > >
> > > > > > > pointer = current component;
> > > > > > > while(parent != null) do {
> > > > > > >     if (any siblings of pointer that are Transparent?)
> > > > > > >         resolved, set parent to the transparent sibling
> > > > > > >     else
> > > > > > >         move pointer to parent of current component
> > > > > > > }
> > > > > > >
> > > > > > > The usecase is still there: if you have some tag that surrounds
> the
> > > > > > > <wicket:child>. We can't support it properly without solving
> this
> > > > > > > problem.
> > > > > > >
> > > > > > > Martijn
> > > > > > >
> > > > > > >
> > > > > > > On 9/12/06, Eelco Hillenius < [EMAIL PROTECTED]> wrote:
> > > > > > > > >         new Label(getMyBorder(), "label", ..);
> > > > > > > >
> > > > > > > > Ah, yeah, of course.
> > > > > > > >
> > > > > > > > Eelco
> > > > > > > >
> > > > > > > >
> > >
> -------------------------------------------------------------------------
> > > > > > > > Using Tomcat but need to do more? Need to support web
> services,
> > > security?
> > > > > > > > Get stuff done quickly with pre-integrated technology to make
> your
> > > job easier
> > > > > > > > Download IBM WebSphere Application Server v.1.0.1 based on
> Apache
> > > Geronimo
> > > > > > > >
> > >
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > > > > > >
> _______________________________________________
> > > > > > > > Wicket-develop mailing list
> > > > > > > > Wicket-develop@lists.sourceforge.net
> > > > > > > >
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Download Wicket 1.2.2 now! New Ajax components: Tree, TreeTable
> and
> > > ModalWindow
> > > > > > > -- http://wicketframework.org
> > > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Download Wicket 1.2.2 now! New Ajax components: Tree, TreeTable
> and
> > > ModalWindow
> > > > > > -- http://wicketframework.org
> > > > > >
> > > > > >
> > >
> -------------------------------------------------------------------------
> > > > > > Using Tomcat but need to do more? Need to support web services,
> > > security?
> > > > > > Get stuff done quickly with pre-integrated technology to make your
> job
> > > easier
> > > > > > Download IBM WebSphere Application Server v.1.0.1 based on Apache
> > > Geronimo
> > > > > >
> > >
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > > > > _______________________________________________
> > > > > > Wicket-develop mailing list
> > > > > > Wicket-develop@lists.sourceforge.net
> > > > > >
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > > > > >
> > > > >
> > > > >
> > >
> -------------------------------------------------------------------------
> > > > > Using Tomcat but need to do more? Need to support web services,
> > > security?
> > > > > Get stuff done quickly with pre-integrated technology to make your
> job
> > > easier
> > > > > Download IBM WebSphere Application Server v.1.0.1 based on Apache
> > > Geronimo
> > > > >
> > >
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > > > _______________________________________________
> > > > > Wicket-develop mailing list
> > > > > Wicket-develop@lists.sourceforge.net
> > > > >
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > > > >
> > > >
> > > >
> > > > --
> > > > Download Wicket 1.2.2 now! New Ajax components: Tree, TreeTable and
> > > ModalWindow
> > > > -- http://wicketframework.org
> > > >
> > > >
> > >
> -------------------------------------------------------------------------
> > > > Using Tomcat but need to do more? Need to support web services,
> security?
> > > > Get stuff done quickly with pre-integrated technology to make your job
> > > easier
> > > > Download IBM WebSphere Application Server v.1.0.1 based on Apache
> Geronimo
> > > >
> > >
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > > _______________________________________________
> > > > Wicket-develop mailing list
> > > > Wicket-develop@lists.sourceforge.net
> > > >
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > > >
> > >
> > >
> > >
> -------------------------------------------------------------------------
> > > Using Tomcat but need to do more? Need to support web services,
> security?
> > > Get stuff done quickly with pre-integrated technology to make your job
> > > easier
> > > Download IBM WebSphere Application Server v.1.0.1 based on Apache
> Geronimo
> > >
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > >
> > > _______________________________________________
> > > Wicket-develop mailing list
> > > Wicket-develop@lists.sourceforge.net
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > >
> > >
> > >
> >
> >
> > --
> > Download Wicket 1.2.2 now! New Ajax components: Tree, TreeTable and
> ModalWindow
> > -- http://wicketframework.org
> >
> >
> -------------------------------------------------------------------------
> > Using Tomcat but need to do more? Need to support web services, security?
> > Get stuff done quickly with pre-integrated technology to make your job
> easier
> > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> >
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > _______________________________________________
> > Wicket-develop mailing list
> > Wicket-develop@lists.sourceforge.net
> >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> >
>
>
> -------------------------------------------------------------------------
> Using Tomcat but need to do more? Need to support web services, security?
> Get stuff done quickly with pre-integrated technology to make your job
> easier
> Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
>
> _______________________________________________
> Wicket-develop mailing list
> Wicket-develop@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
>
>
>

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Wicket-develop mailing list
Wicket-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to