[ http://issues.apache.org/struts/browse/WW-1328?page=comments#action_37601 
] 

Don Brown commented on WW-1328:
-------------------------------

Is the fix as simple as s/xhtml/\${parameters.theme}/ ?  If I don't hear 
otherwise, I'll make the change.

> xhtml theme is hardcoded to use xhtml/controlfooter for all templates
> ---------------------------------------------------------------------
>
>          Key: WW-1328
>          URL: http://issues.apache.org/struts/browse/WW-1328
>      Project: Struts Action 2
>         Type: Bug

>   Components: Views
>     Versions: WW 2.2.2
>     Reporter: Nick Hill
>      Fix For: 2.0.0

>
> If you look at one of the xhtml themes, for example, text.ftl, it is hard 
> coded to use the xhtml controlfooter. This poses a problem for overridding 
> the theme.
> Example xhtml/text.ftl:
> <#include "/${parameters.templateDir}/${parameters.theme}/controlheader.ftl" 
> />
> <#include "/${parameters.templateDir}/simple/text.ftl" />
> <#include "/${parameters.templateDir}/xhtml/controlfooter.ftl" />
> Notice the controlfooter does not use ${parameters.theme} but rather is hard 
> coded to xhtml. 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to