Changeset 203


Ignore:
Timestamp:
09/24/13 13:46:45 (5 years ago)
Author:
nhoyt
Message:

In the Rule Results table (Rule Category or WCAG Guideline views), replaced the three columns V/W, MC and Pass, which showed the number of node results, with a single column labeled Result, which shows the RuleResultValue?: V, W, MC or P.

Also changed the default sort order for the table to be (1) Result value (2) WCAG level and (3) Required.

Note: Method on FilteredRuleResult? should be changed from getResult to getResultValue.

Location:
trunk/proto-14/chrome
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/proto-14/chrome/content/view-category.js

    r200 r203  
    2424ainspectorSidebar.ruleResultConst = { 
    2525  DEFAULT_ORDER: "descending", 
    26   DEFAULT_COLUMN: "ainspector-level-col", 
    27   SORT_COL_2: "ainspector-level-col", 
    28   SORT_COL_3: "ainspector-required-col", 
    29   SORT_COL_4: "ainspector-viol-col" 
     26  DEFAULT_COLUMN: "ainspector-result-col", 
     27  SORT_COL_2: "ainspector-result-col", 
     28  SORT_COL_3: "ainspector-level-col", 
     29  SORT_COL_4: "ainspector-required-col" 
    3030} 
    3131 
     
    4141  var nls = ainspectorSidebar.nlsProperties; 
    4242  return { 
    43     LEVEL_A:     nls.getString("rule.level.a"), 
    44     LEVEL_AA:    nls.getString("rule.level.aa"), 
    45     LEVEL_AAA:   nls.getString("rule.level.aaa"), 
    46     REQUIRED_Y:  nls.getString("rule.required.yes"), 
    47     REQUIRED_N:  nls.getString("rule.required.no"), 
    48     NO_RESULTS:  nls.getString("rule.no-results") 
     43    VIOLATION:    nls.getString("rule.result.violation"), 
     44    WARNING:      nls.getString("rule.result.warning"), 
     45    MANUAL_CHECK: nls.getString("rule.result.manual-check"), 
     46    PASS:         nls.getString("rule.result.pass"), 
     47    LEVEL_A:      nls.getString("rule.level.a"), 
     48    LEVEL_AA:     nls.getString("rule.level.aa"), 
     49    LEVEL_AAA:    nls.getString("rule.level.aaa"), 
     50    REQUIRED_Y:   nls.getString("rule.required.yes"), 
     51    REQUIRED_N:   nls.getString("rule.required.no"), 
     52    NO_RESULTS:   nls.getString("rule.no-results") 
    4953  } 
    5054}; 
     
    6569  for (var i = 0; i < resultsCount; i++) { 
    6670    var filteredResult = filteredRuleResults[i]; 
    67     var summaryData = filteredResult.getNodeResultSummary(); 
    68     var hasResults = summaryData.hasResults(); 
     71    var hasResults = filteredResult.hasResults(); 
    6972    table.push({ 
    7073      "ainspector-rule-col":     filteredResult.getRuleSummary(), 
     74      "ainspector-result-col":   hasResults ? filteredResult.getRuleResult() : -1, 
    7175      "ainspector-level-col":    filteredResult.getWCAG20LevelConstant(), 
    7276      "ainspector-required-col": filteredResult.isRuleRequired(), 
    7377      "ainspector-wcag-sc-col":  filteredResult.getPrimarySuccessCriterion().id, 
    74       "ainspector-viol-col":     hasResults ? summaryData.failures : -1, 
    75       "ainspector-pass-col":     hasResults ? summaryData.passed : -1, 
    76       "ainspector-mc-col":       hasResults ? summaryData.manual_checks : -1, 
    7778      filteredResult:            filteredResult 
    7879    }); 
     
    109110 
    110111    switch (col.id) { 
     112      case "ainspector-result-col": 
     113        switch (value) { 
     114          case OpenAjax.a11y.RESULT_VALUE.VIOLATION: 
     115            return resultStrings.VIOLATION; 
     116            break; 
     117          case OpenAjax.a11y.RESULT_VALUE.WARNING: 
     118            return resultStrings.WARNING; 
     119            break; 
     120          case OpenAjax.a11y.RESULT_VALUE.MANUAL_CHECK: 
     121            return resultStrings.MANUAL_CHECK; 
     122            break; 
     123          case OpenAjax.a11y.RESULT_VALUE.PASS: 
     124            return resultStrings.PASS; 
     125            break; 
     126        } 
     127        break; 
     128 
    111129      case "ainspector-level-col": 
    112130        switch (value) { 
     
    136154  this.getCellProperties = function (row, col, props) { 
    137155    switch (col.id) { 
     156      case "ainspector-result-col": 
    138157      case "ainspector-level-col": 
    139158      case "ainspector-required-col": 
    140159      case "ainspector-wcag-sc-col": 
    141160        props.AppendElement(atomService.getAtom("inner")); 
    142         break; 
    143       case "ainspector-viol-col": 
    144       case "ainspector-mc-col": 
    145       case "ainspector-pass-col": 
    146         props.AppendElement(atomService.getAtom("numeric")); 
    147161        break; 
    148162    } 
  • trunk/proto-14/chrome/content/view-category.xul

    r182 r203  
    4141      persist="sortDirection sortResource" 
    4242      sortDirection="descending" 
    43       sortResource="ainspector-level-col" 
     43      sortResource="ainspector-result-col" 
    4444      onselect="ainspectorSidebar.updateSelectedRuleInfo(this)" 
    4545      ondblclick="ainspectorSidebar.handleRCTreeEvent(event)" 
     
    5151          label="&ai-treecol.rule;" 
    5252          tooltiptext="&ai-treecol.ruleTooltip;" 
     53          persist="width" 
     54          onclick="ainspectorSidebar.sortRulesTable(this)" 
     55          class="sortDirectionIndicator"/> 
     56        <splitter class="tree-splitter"/> 
     57        <treecol id="ainspector-result-col" 
     58          width="56" minwidth="56" maxwidth="60" 
     59          ignoreincolumnpicker="true" 
     60          label="&ai-treecol.result;" 
     61          style="text-align: left; padding-right: 16px" 
     62          tooltiptext="&ai-treecol.resultTooltip;" 
    5363          persist="width" 
    5464          onclick="ainspectorSidebar.sortRulesTable(this)" 
     
    8191          style="text-align: left; padding-right: 16px" 
    8292          tooltiptext="&ai-treecol.wcagSCTooltip;" 
    83           persist="hidden width" 
    84           onclick="ainspectorSidebar.sortRulesTable(this)" 
    85           class="sortDirectionIndicator"/> 
    86         <splitter class="tree-splitter"/> 
    87         <treecol id="ainspector-viol-col" 
    88           width="44" minwidth="44" maxwidth="60" 
    89           ignoreincolumnpicker="true" 
    90           style="text-align: right; padding-right: 16px" 
    91           label="&ai-treecol.viol;" 
    92           tooltiptext="&ai-treecol.violTooltip;" 
    93           persist="hidden width" 
    94           onclick="ainspectorSidebar.sortRulesTable(this)" 
    95           class="sortDirectionIndicator"/> 
    96         <splitter class="tree-splitter"/> 
    97         <treecol id="ainspector-mc-col" 
    98           width="44" minwidth="44" maxwidth="60" 
    99           style="text-align: right; padding-right: 16px" 
    100           label="&ai-treecol.manualChecks;" 
    101           tooltiptext="&ai-treecol.manualChecksTooltip;" 
    102           persist="hidden width" 
    103           onclick="ainspectorSidebar.sortRulesTable(this)" 
    104           class="sortDirectionIndicator"/> 
    105         <splitter class="tree-splitter"/> 
    106         <!-- "Pass" column is hidden by default --> 
    107         <treecol id="ainspector-pass-col" hidden="true" 
    108           width="44" minwidth="44" maxwidth="60" 
    109           style="text-align: right; padding-right: 16px" 
    110           label="&ai-treecol.pass;" 
    111           tooltiptext="&ai-treecol.passTooltip;" 
    11293          persist="hidden width" 
    11394          onclick="ainspectorSidebar.sortRulesTable(this)" 
  • trunk/proto-14/chrome/locale/en-US/ai-sidebar.dtd

    r202 r203  
    9595<!ENTITY ai-treecol.rule                    "Rule"> 
    9696<!ENTITY ai-treecol.ruleTooltip             "Rule title"> 
     97<!ENTITY ai-treecol.result                  "Result"> 
     98<!ENTITY ai-treecol.resultTooltip           "Evaluation result"> 
    9799<!ENTITY ai-treecol.level                   "Lev."> 
    98100<!ENTITY ai-treecol.levelTooltip            "WCAG level"> 
  • trunk/proto-14/chrome/locale/en-US/ai-sidebar.properties

    r202 r203  
    4646info.wcag-level        = WCAG 2.0 Level\u0020 
    4747 
    48 rule.level.a*  = A* 
     48rule.result.violation    = V 
     49rule.result.warning      = W 
     50rule.result.manual-check = MC 
     51rule.result.pass         = P 
     52 
    4953rule.level.a   = A 
    5054rule.level.aa  = AA 
Note: See TracChangeset for help on using the changeset viewer.