[jira] Resolved: (TRINIDAD-20) SKINNING - DO NOT SHORTEN STYLECLASSES THAT START WITH P_AF (THE STATE PREFIX)

["Jeanne Waldman (JIRA)" <de...@myfaces.apache.org>]

     [ https://issues.apache.org/jira/browse/TRINIDAD-20?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jeanne Waldman resolved TRINIDAD-20.
------------------------------------

    Resolution: Fixed

> SKINNING - DO NOT SHORTEN STYLECLASSES THAT START WITH P_AF (THE STATE PREFIX)
> ------------------------------------------------------------------------------
>
>                 Key: TRINIDAD-20
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-20
>             Project: MyFaces Trinidad
>          Issue Type: Improvement
>          Components: Skinning
>    Affects Versions: 1.0.1-incubating-plugins-SNAPSHOT
>            Reporter: Jeanne Waldman
>            Priority: Minor
>             Fix For: 1.0.1-incubating-plugins-SNAPSHOT
>
>
> Sometimes skin developers accidentally or as a workaround for other bugs will
> use .p_AFFoo styleclasses in their css file instead of :foo.
> e.g., af|inputText.p_AFDisabled instead of af|inputText:disabled
> The result is that the p_AFFoo styleclass is compressed when it shouldn't be.
> Developers rely on .p_AFFoo to not be compressed when it renders on the dom node
> so that they can easily add/remove the styleclass on the client javascript code when the state changes.
>  When ONE instance of .p_AFFoo exists in the css/xss file, then p_AFFoo is compressed everywhere it
>  is used. Then adding/removing "p_AFFoo" wll not work because it will be compressed to something
> like "x14"
> I plan to not compress any styleclasses that start with p_AF (our state
> prefix - SkinSelectors.STATE_PREFIX). This way things won't break if someone
> screws it up. 

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