On 11/15/05, Johan Compagner <[EMAIL PROTECTED]> wrote:
> No ofcourse i don't want to add isFullRender() that test that
> markupStreamPosition
> That markup stream position shouldn't be tested at all!!
> That is the problem.
>
> i want something like: RequestCycle.isFullRender() that is set when a page
> starts to render itself.
>
>
> We are now always setting the markupstream position in the markupstream
> (after the first time)
> I want to turn that around. Always get the markupstream position from the
> stream but only set it
> if we are in partial render.
>
> Then the current problems people have when using the same tag twice is not a
> problem
> Resetting the markupstream postion code doesn't also have to be there
> anymore. Because it always goes right.
> (if you do a partial render before a full render then it will go wrong
> anyway/always)
>
> And ajax partial render also works for components that are used twice. Only
> the last markupstream position is used

How? Each component is only able to store one markup position. How can
a single component be re-rendered on two different positions?

> And if people can live with that then they can use that if they want to.
> Fine by me
> And maybe for 90% of the usecases that component wouldn't even be used in
> partial rendering.
>

IMO is bad design to tell users that in case A you can but in case B
you can not. What about application which get improved with Ajax
components. You than have to review you whole application. IMO a bad
idea. I still prefer it to be simple 1:1

The idea of using a isFullRender flag is more explicit, though. Makes
reading the code easier

Juergen


-------------------------------------------------------
This SF.Net email is sponsored by the JBoss Inc.  Get Certified Today
Register for a JBoss Training Course.  Free Certification Exam
for All Training Attendees Through End of 2005. For more info visit:
http://ads.osdn.com/?ad_idv28&alloc_id845&op=click
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to