[OpenLayers-Commits] r4248 - trunk/openlayers/lib/OpenLayers/Popup

commits at openlayers.org commits at openlayers.org
Wed Sep 12 15:25:21 EDT 2007


Author: euzuro
Date: 2007-09-12 15:25:20 -0400 (Wed, 12 Sep 2007)
New Revision: 4248

Modified:
   trunk/openlayers/lib/OpenLayers/Popup/AnchoredBubble.js
Log:
coding standards -- mostly ND comment style corrections and lines longer than 79char

Modified: trunk/openlayers/lib/OpenLayers/Popup/AnchoredBubble.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Popup/AnchoredBubble.js	2007-09-12 19:22:33 UTC (rev 4247)
+++ trunk/openlayers/lib/OpenLayers/Popup/AnchoredBubble.js	2007-09-12 19:25:20 UTC (rev 4248)
@@ -11,7 +11,8 @@
  * Inherits from: 
  *  - <OpenLayers.Popup.Anchored>
  */
-OpenLayers.Popup.AnchoredBubble = OpenLayers.Class(OpenLayers.Popup.Anchored, {
+OpenLayers.Popup.AnchoredBubble = 
+  OpenLayers.Class(OpenLayers.Popup.Anchored, {
 
     /**
      * Property: rounded
@@ -29,7 +30,7 @@
      * contentHTML - {String}
      * anchor - {Object} Object to which we'll anchor the popup. Must expose 
      *     a 'size' (<OpenLayers.Size>) and 'offset' (<OpenLayers.Pixel>) 
-     *     (Note that this is generally an <OpenLayers.Icon>)
+     *     (Note that this is generally an <OpenLayers.Icon>).
      * closeBox - {Boolean}
      */
     initialize:function(id, lonlat, size, contentHTML, anchor, closeBox) {
@@ -43,7 +44,7 @@
      * px - {<OpenLayers.Pixel>}
      * 
      * Returns:
-     * {DOMElement} Reference to a div that contains the drawn popup
+     * {DOMElement} Reference to a div that contains the drawn popup.
      */
     draw: function(px) {
         
@@ -101,7 +102,8 @@
         if (this.div != null) {
             if (this.contentDiv != null) {
                 this.div.style.background = "transparent";
-                OpenLayers.Rico.Corner.changeColor(this.contentDiv, this.backgroundColor);
+                OpenLayers.Rico.Corner.changeColor(this.contentDiv, 
+                                                   this.backgroundColor);
             }
         }
     },  
@@ -119,14 +121,14 @@
         
         if (this.div != null) {
             if (this.contentDiv != null) {
-            OpenLayers.Rico.Corner.changeOpacity(this.contentDiv, this.opacity);
+                OpenLayers.Rico.Corner.changeOpacity(this.contentDiv, 
+                                                     this.opacity);
             }
         }
     },  
  
     /** 
      * Method: setBorder
-     * 
      * Always sets border to 0. Bubble Popups can not have a border.
      * 
      * Parameters:
@@ -138,12 +140,10 @@
  
     /** 
      * Method: setRicoCorners
-     * 
      * Update RICO corners according to the popup's current relative postion.
      *  
      * Parameters:
-     * firstTime - {Boolean} Is this the first time the corners are being
-     *                       rounded?
+     * firstTime - {Boolean} This the first time the corners are being rounded.
      */
     setRicoCorners:function(firstTime) {
     
@@ -167,7 +167,7 @@
      * Method: getCornersToRound
      *  
      * Returns:
-     * {String} The proper corners string ("tr tl bl br") for rico to round
+     * {String} The proper corners string ("tr tl bl br") for rico to round.
      */
     getCornersToRound:function() {
 
@@ -185,7 +185,7 @@
 
 /**
  * Constant: CORNER_SIZE
- * {Integer} 5. Border space for the RICO corners
+ * {Integer} 5. Border space for the RICO corners.
  */
 OpenLayers.Popup.AnchoredBubble.CORNER_SIZE = 5;
 



More information about the Commits mailing list