[TRINIDAD] New element in trinidad-skins.xml /SkinImpl/SkinAddition

[Jeanne Waldman]

Hi, I have a new issue I need to resolve and I wanted to run by my solution -- https://issues.apache.org/jira/browse/TRINIDAD-728 "support for el to be used in a skin to bind to other translation data sources" Currently, a SkinExtension and SkinAddition can have resource bundles associated with them so that a person can "skin" text. We have customers who want to use a Map that is EL-accessible instead of a ResourceBundle. I'd like to add a  'translation-map' element to the <skin> and <skin-addition> elements in trinidad-skins.xml. I'd add new constructors to SkinExtension and SkinAddition to accept a translationMap ValueExpression. Let me know what you think and if you think 'translation-map' is a good name for the new element. See below for an example. Thanks, Jeanne from trinidad-skins.xml:     <skin>         <id>             purple.desktop         </id>         <family>             purple         </family>         <render-kit-id>             org.apache.myfaces.trinidad.desktop         </render-kit-id>         <style-sheet-name>             skins/purple/purpleSkin.css         </style-sheet-name>         <bundle-name>             org.apache.myfaces.trinidaddemo.resource.SkinBundle         </bundle-name>     </skin>     <!-- You can extend any skin you want. Here we want the purple     skin, but with a bigger font size -->     <skin>         <id>             purpleBigFont.desktop         </id>         <family>             purpleBigFont         </family>         <extends>             purple.desktop         </extends>         <render-kit-id>             org.apache.myfaces.trinidad.desktop         </render-kit-id>         <style-sheet-name>             skins/purple/purpleBigFontSkin.css         </style-sheet-name>         <translation-map>#{skinTranslationMap.contents}</translation-map>     </skin>