[jboss-svn-commits] JBoss Portal SVN: r5240 - trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic

[jboss-svn-commits at lists.jboss.org]

Author: roy.russo at jboss.com
Date: 2006-09-21 17:01:47 -0400 (Thu, 21 Sep 2006)
New Revision: 5240

Modified:
   trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaRegionRenderer.java
   trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaWindowRenderer.java
Log:
- remove unnecessary regions
- use window names as div identifiers

Modified: trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaRegionRenderer.java
===================================================================
--- trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaRegionRenderer.java	2006-09-21 15:15:29 UTC (rev 5239)
+++ trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaRegionRenderer.java	2006-09-21 21:01:47 UTC (rev 5240)
@@ -61,14 +61,8 @@
    public void render(RenderContext renderContext, Region region,
                       Map portletContexts, Map windowResults) throws RenderException
    {
-      renderContext.getMarkupFragment().append(
-            "<div id=\"regionDD" + renderContext.getRegionID() + "\">\n");
-
       delegate.render(renderContext, region, portletContexts, windowResults);
 
-      renderContext.getMarkupFragment().append("</div>\n");
-
-
       if(portletPos.get(renderContext.getRegionID()) != null
          && ((Integer) portletPos.get(renderContext.getRegionID())).intValue() > 0)
       {

Modified: trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaWindowRenderer.java
===================================================================
--- trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaWindowRenderer.java	2006-09-21 15:15:29 UTC (rev 5239)
+++ trunk/theme/src/main/org/jboss/portal/theme/impl/render/dynamic/DynaWindowRenderer.java	2006-09-21 21:01:47 UTC (rev 5240)
@@ -65,8 +65,7 @@
             "portlet-dnd-" + renderContext.getRegionID() + "-"
             + portletNumber, portletContext.getWindowName());
 
-      renderContext.getMarkupFragment().append("<div id=\"portlet-dnd-").append(renderContext.getRegionID())
-            .append("-").append(portletNumber).append("\">\n");
+      renderContext.getMarkupFragment().append("<div id=\"").append(portletContext.getWindowName()).append("\">\n");
 
       delegate.render(renderContext, portletContext, result);