[
https://issues.apache.org/jira/browse/TRINIDAD-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12496420
]
Jeanne Waldman commented on TRINIDAD-20:
----------------------------------------
svn 538754
Changed FileSystemStyleCache's code which generates the shortened style class
map to not include styles that start with RichSelectors.STATE_PREFIX in its map.
> 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.
