You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ma...@apache.org on 2008/12/08 08:57:41 UTC

svn commit: r724279 - in /myfaces/trinidad/trunk: src/site/xdoc/skin-selectors.xml trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelPopupRenderer.java

Author: matzew
Date: Sun Dec  7 23:57:41 2008
New Revision: 724279

URL: http://svn.apache.org/viewvc?rev=724279&view=rev
Log:
TRINIDAD-1012 - SKinning for PanelPopup trigger

Thanks to Vimal Kumar Gupta for the patch

Modified:
    myfaces/trinidad/trunk/src/site/xdoc/skin-selectors.xml
    myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelPopupRenderer.java

Modified: myfaces/trinidad/trunk/src/site/xdoc/skin-selectors.xml
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/src/site/xdoc/skin-selectors.xml?rev=724279&r1=724278&r2=724279&view=diff
==============================================================================
--- myfaces/trinidad/trunk/src/site/xdoc/skin-selectors.xml (original)
+++ myfaces/trinidad/trunk/src/site/xdoc/skin-selectors.xml Sun Dec  7 23:57:41 2008
@@ -2234,6 +2234,16 @@
             <th>Description</th>
           </tr>
           <tr>
+            <td>af|panelPopup::trigger</td>
+            <td>Styles the trigger facet of the panelPopup.  Use this selector to control border and margin/padding 
+            around the trigger.</td>
+          </tr>
+          <tr>
+            <td>af|panelPopup::link</td>
+            <td>Styles the link element of the trigger.  Use this selector to control border, color and background 
+            around the trigger-link.</td>
+          </tr>
+          <tr>
             <td>af|panelPopup::container</td>
             <td>Styles the outer element of the popup.  Use this selector to control border and margin/padding 
             around the entire popup.</td>

Modified: myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelPopupRenderer.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelPopupRenderer.java?rev=724279&r1=724278&r2=724279&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelPopupRenderer.java (original)
+++ myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/PanelPopupRenderer.java Sun Dec  7 23:57:41 2008
@@ -150,6 +150,7 @@
     writer.startElement(XhtmlConstants.SPAN_ELEMENT, component);
     writer.writeAttribute(XhtmlConstants.ID_ATTRIBUTE + _POPUP_TRIGGER_ID_SUFFIX, 
         XhtmlUtils.getJSIdentifier(clientId), null);
+    renderStyleClass(context, arc, SkinSelectors.AF_PANEL_POPUP_TRIGGER_STYLE_CLASS);
   
     renderTrigger(context, arc, component, bean);
     
@@ -349,6 +350,12 @@
       return PanelPopupRenderer.this.getStyleClass(bean);
     }
     
+    @Override
+    protected String getDefaultStyleClass(FacesBean bean)
+    {
+      return SkinSelectors.AF_PANEL_POPUP_LINK_STYLE_CLASS;
+    }
+    
     protected String getTriggerScript(FacesBean bean, String componentId)
     {
       String clientId = XhtmlUtils.getJSIdentifier(componentId);