You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-scm@portals.apache.org by es...@apache.org on 2008/03/06 06:41:24 UTC

svn commit: r634172 - in /portals/pluto/trunk: pluto-portal-driver/src/main/java/org/apache/pluto/driver/tags/PortletWindowStateAnchorTag.java pluto-portal/src/main/webapp/WEB-INF/themes/portlet-skin.jsp

Author: esm
Date: Wed Mar  5 21:41:23 2008
New Revision: 634172

URL: http://svn.apache.org/viewvc?rev=634172&view=rev
Log:
[PLUTO-475]: Cleaning up some HTML cruft.  Thanks Benjamin Gould! (merged from pluto-1.1.x 634171)

Modified:
    portals/pluto/trunk/pluto-portal-driver/src/main/java/org/apache/pluto/driver/tags/PortletWindowStateAnchorTag.java
    portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/themes/portlet-skin.jsp

Modified: portals/pluto/trunk/pluto-portal-driver/src/main/java/org/apache/pluto/driver/tags/PortletWindowStateAnchorTag.java
URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal-driver/src/main/java/org/apache/pluto/driver/tags/PortletWindowStateAnchorTag.java?rev=634172&r1=634171&r2=634172&view=diff
==============================================================================
--- portals/pluto/trunk/pluto-portal-driver/src/main/java/org/apache/pluto/driver/tags/PortletWindowStateAnchorTag.java (original)
+++ portals/pluto/trunk/pluto-portal-driver/src/main/java/org/apache/pluto/driver/tags/PortletWindowStateAnchorTag.java Wed Mar  5 21:41:23 2008
@@ -100,7 +100,7 @@
             tag.append(ToolTips.forWindowState(new WindowState(state)));
             tag.append("\" ");
             tag.append("href=\"" + portalUrl.toString() + "\">");
-            tag.append("<span class=\"" + state + "\"></span>");
+            tag.append("<span class=\"" + state + "\">");
             tag.append("</span></a>");
 
             // Print the mode anchor tag.

Modified: portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/themes/portlet-skin.jsp
URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/themes/portlet-skin.jsp?rev=634172&r1=634171&r2=634172&view=diff
==============================================================================
--- portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/themes/portlet-skin.jsp (original)
+++ portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/themes/portlet-skin.jsp Wed Mar  5 21:41:23 2008
@@ -23,7 +23,7 @@
 <pluto:portlet portletId="${portlet}">
 
   <!-- Assemble the rendering result -->
-  <div class="portlet" id='<c:out value="${portlet}"/>'>
+  <div class="portlet">
     <div class="header">
       <!-- Portlet Mode Controls -->
       <pluto:modeAnchor portletMode="view"/>
@@ -33,9 +33,6 @@
       <pluto:windowStateAnchor windowState="minimized"/>
       <pluto:windowStateAnchor windowState="maximized"/>
       <pluto:windowStateAnchor windowState="normal"/>
-      <a href="<pluto:url windowState="minimized"/>"><span class="min"></span></a>
-      <a href="<pluto:url windowState="maximized"/>"><span class="max"></span></a>
-      <a href="<pluto:url windowState="normal"/>"><span class="norm"></span></a>
       <!-- Portlet Title -->
       <h2 class="title"><pluto:title/></h2>
     </div>