Changeset 212


Ignore:
Timestamp:
10/01/13 19:42:23 (6 years ago)
Author:
nhoyt
Message:

element-results.js

  • removed ainspector-element-col from case in getCellProperties that returns "inner" to fix formatting problem

highlight.js

  • deleted highlightCacheItems fn. - not needed by AInspector Sidebar
  • fixed logic in highlightNodeResults fn. that was unintentionally filtering out nodes with Website scope
  • add "embed" to elements_without_content array in inserDIV fn.

view-details.xul

  • fixed text-align problem in ainspector-order-col: now text-align: right for numeric data

ai-sidebar.css

  • changed treechildren::-moz-tree-cell(numeric) property to increase padding-right amount
Location:
trunk/proto-14/chrome
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/proto-14/chrome/content/element-results.js

    r209 r212  
    177177  this.getCellProperties = function (row, col, props) { 
    178178    switch (col.id) { 
    179       case "ainspector-element-col": 
    180179      case "ainspector-elem-result-col": 
    181180      case "ainspector-action-col": 
  • trunk/proto-14/chrome/content/highlight/highlight.js

    r211 r212  
    111111 
    112112  /** 
    113    * @function highlightCacheItems 
    114    * 
    115    * @desc Highlight a single cache item or a list of cache items. Uses worst 
    116    *       node result of each cache item to style highlight. 
    117    * 
    118    * @param {Object} document - the target DOM 
    119    * @param {Object} item - cache item object or a list of cache item objects 
    120    *        (NOTE: can be simple or set of dom element) to highlight 
    121    */ 
    122   highlightCacheItems: function (document, item) { 
    123  
    124     var VISIBILITY = OpenAjax.a11y.VISIBILITY; 
    125     this.removeHighlight(document); 
    126     if (typeof item !== 'object') return; 
    127  
    128     var cache_items = []; 
    129  
    130     if (typeof item.dom_element === 'object' || typeof item.type === 'number') { 
    131       cache_items.push(item) 
    132     } 
    133     else { 
    134       cache_items = item; 
    135     } 
    136  
    137     var cache_items_len = cache_items.length; 
    138     var off_screen_elements = []; 
    139  
    140     var v = 0; 
    141     var w = 0; 
    142     var pp = 0; 
    143     var m = 0; 
    144     var hh = 0; 
    145  
    146     for (var i = 0; i < cache_items_len; i++) { 
    147       if (typeof cache_item[i] !== 'object') continue; 
    148  
    149       var cache_item = cache_items[i]; 
    150  
    151       var de = (typeof cache_item.dom_element === 'object') ? 
    152                cache_item.dom_element : cache_item; 
    153  
    154       var style = this.getWorstResultValueStyle(de); 
    155  
    156       var style_de = (de.type === Node.TEXT_NODE) ? 
    157                      de.parent_element : de; 
    158  
    159       var node     = style_de.node; 
    160       var tag_name = style_de.tag_name; 
    161  
    162       if (node) { 
    163  
    164         // check if the node is off screen or hidden from assistive technologies 
    165         if (style_de.computed_style.is_visible_onscreen === VISIBILITY.HIDDEN) { 
    166  
    167           if (de.rules_violations.length || 
    168               de.rules_warnings.length || 
    169               (this.show_page_manual_check && this.scopePageCount(de.rules_manual_checks)) || 
    170               (this.show_element_manual_check && this.scopeElementCount(de.rules_manual_checks)) || 
    171               (this.show_pass && de.rules_passed.length) || 
    172               (this.show_hidden && de.rules_hidden.length)) { 
    173  
    174             off_screen_elements.push(cache_item); 
    175  
    176             v += (de.rules_violations.length > 0) ? de.rules_violations.length : 0; 
    177             w += (de.rules_warnings.length > 0) ? de.rules_warnings.length : 0; 
    178             p += (this.show_pass && de.rules_passed.length > 0) ? de.rules_passed.length : 0; 
    179             m += (this.show_page_manual_check && de.rules_manual_checks.length > 0) ? 
    180                  de.rules_manual_checks.length : 0; 
    181             h += (this.show_hidden && de.rules_hidden.length) ? de.rules_hidden.length : 0; 
    182           } 
    183  
    184         } 
    185         else { 
    186  
    187           var mc = 0; 
    188           var p = 0; 
    189           var h = 0; 
    190  
    191           if (this.show_page_manual_check) 
    192             mc += this.scopePageCount(de.rules_manual_checks); 
    193           if (this.show_element_manual_check) 
    194             mc += this.scopeElementCount(de.rules_manual_checks); 
    195           if (this.show_pass) 
    196             p = de.rules_pass; 
    197           if (this.show_hidden) 
    198             h = de.rules_hidden; 
    199  
    200           this.insertDIV(document, node, tag_name, style, 
    201                          de.rules_violations, de.rules_warnings, 
    202                          mc, p, h); 
    203         } 
    204       } 
    205     } //end for 
    206  
    207     if (off_screen_elements.length > 0) 
    208       this.showOffScreenCacheItems(off_screen_elements, v, w, m, pp, hh ); 
    209   }, 
    210  
    211   /** 
    212113   * @function highlightNodeResults 
    213114   * 
     
    314215 
    315216            case RESULT_VALUE.MANUAL_CHECK: 
    316               if (this.show_page_manual_check && node_result.isScopePage()) 
    317                 this.insertDIV(document, node, tag_name, this.STYLES.manualChecks, 0, 0, 1, 0, 0); 
    318               if (this.show_element_manual_check && node_result.isScopeElement()) 
    319                 this.insertDIV(document, node, tag_name, this.STYLES.manualChecks, 0, 0, 1, 0, 0); 
     217              if (node_result.isScopeElement() && !this.show_element_manual_check) 
     218                continue; 
     219              if (node_result.isScopePage() && !this.show_page_manual_check) 
     220                continue; 
     221              this.insertDIV(document, node, tag_name, this.STYLES.manualChecks, 0, 0, 1, 0, 0); 
    320222              break; 
    321223 
     
    563465      'area', 
    564466      'dl', 
     467      'embed', 
    565468      'frame', 
    566469      'img', 
     
    594497      node.appendChild(new_div_element); 
    595498    } 
    596  
    597     //  If true, element is aligned with top of scroll area. 
    598     //  If false, it is aligned with bottom. 
    599     /* 
    600     node.scrollIntoView(true); 
    601     if (window.content.scrollY) { 
    602       var offset = window.content.scrollY - 10; 
    603       window.content.scroll(0, offset); 
    604     } 
    605     */ 
    606499  }, 
    607500 
  • trunk/proto-14/chrome/content/view-details.xul

    r208 r212  
    6262                ignoreincolumnpicker="true" 
    6363                label="&ai-treecol.order;" 
    64                 style="text-align: left; padding-right: 16px" 
     64                style="text-align: right; padding-right: 16px" 
    6565                tooltiptext="&ai-treecol.orderTooltip;" 
    6666                persist="width" 
  • trunk/proto-14/chrome/skin/ai-sidebar.css

    r181 r212  
    2929 
    3030treechildren::-moz-tree-cell(numeric) { 
    31   padding-right: 8px; 
     31  padding-right: 16px; 
    3232} 
    3333 
Note: See TracChangeset for help on using the changeset viewer.