ams-tschoening commented on a change in pull request #457:
URL: https://github.com/apache/wicket/pull/457#discussion_r508265392



##########
File path: 
wicket-core/src/main/java/org/apache/wicket/markup/transformer/AbstractTransformerBehavior.java
##########
@@ -26,17 +31,114 @@
 import org.apache.wicket.request.http.WebResponse;
 
 /**
- * A {@link Behavior} which can be added to any component. It allows to 
post-process (transform) the
+ * A {@link Behavior} which can be added to any component, allowing to 
post-process (transform) the
  * markup generated by the component.
- * 
+ * <p>
+ * There's one important limitation with the current implementation: Multiple 
different instances of

Review comment:
       Those 20 lines of history are exactly what I regularly miss when running 
into trouble with Wicket, because without them there's no basic understanding 
of why things are the way they are. When running into this problem, I needed to 
look at the entire GIT-history just to find no reason for the current 
implementation, no clue on how things were supposed to work, for a workaround 
fitting the existing design etc.
   
   The docs often lack important context from my experience, so I'm trying to 
add one here.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to