Sadly it can't (easily) -- the beacon needs to appear before feature libs,
which may be inlined or extern. Creating a new <script> tag avoids having to
deal w/ that conditional logic.

On Wed, Jan 20, 2010 at 11:42 PM, <lind...@inuus.com> wrote:

>
> http://codereview.appspot.com/186253/diff/1/3
> File
>
>
> java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingGadgetRewriter.java
> (right):
>
> http://codereview.appspot.com/186253/diff/1/3#newcode166
>
> java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingGadgetRewriter.java:166:
> injectGadgetBeacon(gadget, head);
> could we avoid an extra script tag here and inject into the
> mainscripttag below?
>
>
> http://codereview.appspot.com/186253/show
>

Reply via email to