[
http://jira.magnolia-cms.com/browse/MAGNOLIA-2438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18722#action_18722
]
Gregory Joseph commented on MAGNOLIA-2438:
------------------------------------------
The patch looks "strange" only because your ide went ahead and reformatted the
code without being asked to do it :)
Actually if you code redo it without the code style change it'd be great, we'd
see where the actual changes are.
> Add "first" to li class on first element, "last" to li class on last element
> of level
> -------------------------------------------------------------------------------------
>
> Key: MAGNOLIA-2438
> URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-2438
> Project: Magnolia
> Issue Type: Improvement
> Components: taglibs
> Affects Versions: 3.6.3
> Reporter: Will Scheidegger
> Assignee: Fabrizio Giustina
> Priority: Minor
> Attachments: SimpleNavigationTag.java, SimpleNavigationTag.patch
>
> Original Estimate: 0.06d
> Remaining Estimate: 0.06d
>
> Often one would like to have the first or last nav element of a level display
> differently, e.g. an additional border. The proposed modification adds a
> "first" to the css class of the first element and a "last" to the last nav
> element of each level, i.e. to the <li> tag.
> For instance this:
> <ul class="level3">
> <li class="leaf"><a href="/magnoliaAuthor/page1.html">Link 1</a></li>
> <li class="leaf"><a href="/magnoliaAuthor/page2.html">Link 2</a></li>
> <li class="leaf"><a href="/magnoliaAuthor/page3.html">Link 3</a></li>
> <li class="leaf"><a href="/magnoliaAuthor/page4.html">Link 4</a></li>
> </ul>
> will be turned into this:
> <ul class="level3">
> <li class="leaf first"><a href="/magnoliaAuthor/page1.html">Link 1</a></li>
> <li class="leaf"><a href="/magnoliaAuthor/page2.html">Link 2</a></li>
> <li class="leaf"><a href="/magnoliaAuthor/page3.html">Link 3</a></li>
> <li class="leaf last"><a href="/magnoliaAuthor/page4.html">Link 4</a></li>
> </ul>
> Note: I did create the patch file as described in
> http://wiki.magnolia-cms.com/display/WIKI/Developer+information but the patch
> file looks strange with many many lines to be removed. Therefore I also added
> the complete, modified Java file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
for list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
----------------------------------------------------------------
