[ 
https://issues.apache.org/jira/browse/TOMAHAWK-1341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12634610#action_12634610
 ] 

Joseph ONeil commented on TOMAHAWK-1341:
----------------------------------------

For cleaner code
 if (_styleLocation != null) setStringProperty(component, 
JSFAttr.STYLE_LOCATION, _styleLocation); 

can be change to 
setStringProperty(component, JSFAttr.STYLE_LOCATION, _styleLocation); 

since null is already checked

> JS Cook Menu stylelocation no longer working on 1.1.7
> -----------------------------------------------------
>
>                 Key: TOMAHAWK-1341
>                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-1341
>             Project: MyFaces Tomahawk
>          Issue Type: Bug
>          Components: JS Cook Menu
>    Affects Versions: 1.1.7
>         Environment: window Jboss Java 5 and tomahawk 1.1.7 for 1.2 jsf
>            Reporter: Joseph ONeil
>         Attachments: HtmlJSCookMenuTag.java
>
>
> The stylelocation in 1.16 allow me to set custom color to the menu this is no 
> longer and instead showing the default color
> <t:jscookMenu layout="hbr" theme="ThemeOffice" 
> styleLocation="/css/jscookmenu" >
> .....
> </t:jscookMenu>
>  Looking at the html source you can see how the href is being created note 
> Trade is the application name
> version 1.1.6 (Link is correct)
> link rel="stylesheet" href="/Trade/css/jscookmenu/ThemeOffice/theme.css" 
> type="text/css" />
> version .1.17
> <link rel="stylesheet" 
> href="/Trade/faces/extensionResource/org.apache.myfaces.renderkit.html.util.MyFacesResourceLoader/12222884/navmenu.jscookmenu.HtmlJSCookMenuRenderer/ThemeOffice/theme.css"
>  type="text/css" />

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to