Ben, that looks like some excellent JSP. Almost no scriptlets, and
that's what counts for me :)

Here's a couple of things I noticed you can change to reduce amount of code:

<jsp:scriptlet>out.println(defaultarrowstyle);</jsp:scriptlet>

should be able to replace with

<c:out value="${defaultarrowstyle}"/>

and

<c:set var="divid">
            ${contentNodeCollectionName}
</c:set>

can be written as

<c:set var="divid" value="${contentNodeCollectionName}"/>


Bah, but now I'm just nit-picking. Add to the wiki please, and add a
link under the Templates section:

http://www.magnolia.info/wiki/Wiki.jsp?page=Magnolia2.1

On 6/26/06, Ben Brock <[email protected]> wrote:
The below code works for me. Let me know if you think it's worth posting
on the wiki. (It may need some tidying up first.)

----------------------------------------------------------------
for list details see
http://www.magnolia.info/en/magnolia/developer.html
----------------------------------------------------------------

Reply via email to