fix bug annotation script and styling

Author: astearns

css3-regions/BugzillaTrackerCSSRegions.js

@@ -48,14 +48,20 @@
                 if ( (documentIssue.bug_status !== serverIssue.bug_status) || 
                     (documentIssue.short_desc !== serverIssue.short_desc) ){  
 
+                    // changes from NEW->ASSIGNED aren't noteworthy
+                    if ( ! (documentIssue.bug_status == "NEW" &&
+                         serverIssue.bug_status == "ASSIGNED") )
                     serverIssue.issue_state = "updated"; 
                 }  
             }
             else{ 
                 // not found in the document. it's a new issue if the bug_status
                 // is not RESOLVED
                 if (serverIssue.bug_status === "RESOLVED") {
-                    serverIssue.issue_state = 'resolved';
+                    //why do we want to show resolved issues that
+                    //have already been removed from the spec?
+
+                    //serverIssue.issue_state = 'resolved';
                 } else {
                     serverIssue.issue_state = "new";
                 }
@@ -82,22 +88,23 @@
 
         for (var issueId in issueList){
 
-            // bind the bug data to the bug template
-            var issueFragment = this.renderIssue(issueList[issueId]);
-            
-            var wrapper = document.createElement("div"); 
-            var trigger = document.createElement("a");
-            trigger.className = "issue-markup-trigger";
-            trigger.href = "#";          
-            trigger.innerHTML = "toggle markup"
-            
             if (issueList[issueId]["issue_state"]){
+
+                // bind the bug data to the bug template
+                var issueFragment = this.renderIssue(issueList[issueId]);
+                
+                var wrapper = document.createElement("div"); 
+                var trigger = document.createElement("a");
+                trigger.className = "issue-markup-trigger";
+                trigger.href = "#";          
+                trigger.innerHTML = "toggle markup"
+                
                 wrapper.setAttribute("data-issue_state", issueList[issueId]["issue_state"]);
+                
+                wrapper.appendChild(trigger);
+                wrapper.appendChild(issueFragment);
+                fragment.appendChild(wrapper); 
             }     
-            
-            wrapper.appendChild(trigger);
-            wrapper.appendChild(issueFragment);
-            fragment.appendChild(wrapper); 
         }
 
         var issueListContainer = document.querySelector("#issue-list"); 

css3-regions/Overview.html

@@ -149,9 +149,13 @@
               background: #eee;      
               text-decoration: none;    
               color: #444;
-              position: absolute;
+              /* this was causing all the trigger
+                 links to stack up in one spot,
+                 even though the wrapper is
+                 relative position */
+              /*position: absolute;
               right: 10px;
-              top: 10px;  
+              top: 10px; */
               z-index: 1;
           }  
 
@@ -224,13 +228,13 @@
 
    <h1 id=css-regions-module>CSS Regions Module Level 3</h1>
 
-   <h2 class="no-num no-toc" id=longstatus-date>Editor's Draft 12 April 2012</h2>
+   <h2 class="no-num no-toc" id=longstatus-date>Editor's Draft 14 April 2012</h2>
 
    <dl>
     <dt>This version:
 
     <dd><a
-     href="http://www.w3.org/TR/2012/ED-css3-regions-20120412/">http://www.w3.org/csswg/css3-regions</a>
+     href="http://www.w3.org/TR/2012/ED-css3-regions-20120414/">http://www.w3.org/csswg/css3-regions</a>
 
     <dt>Latest version:
 

css3-regions/Overview.src.html

@@ -147,9 +147,13 @@
               background: #eee;      
               text-decoration: none;    
               color: #444;
-              position: absolute;
+              /* this was causing all the trigger
+                 links to stack up in one spot,
+                 even though the wrapper is
+                 relative position */
+              /*position: absolute;
               right: 10px;
-              top: 10px;  
+              top: 10px; */
               z-index: 1;
           }