diff --git a/ui/css/perf.css b/ui/css/perf.css
index a372835a476..a8dd9d74114 100644
--- a/ui/css/perf.css
+++ b/ui/css/perf.css
@@ -308,6 +308,14 @@ th {
.compare-table tr .result-links a,
.compare-table tr .result-links button {
color: #23527c;
+ opacity: 0.5;
+}
+
+.compare-table tr .result-links a:hover,
+.compare-table tr .result-links button:hover,
+.compare-table tr .result-links a:focus,
+.compare-table tr .result-links button:focus {
+ opacity: 1;
}
.compare-table tr:hover .retrigger-btn {
diff --git a/ui/perfherder/ProgressBar.jsx b/ui/perfherder/ProgressBar.jsx
index 0372bf3947b..338624ff4f2 100644
--- a/ui/perfherder/ProgressBar.jsx
+++ b/ui/perfherder/ProgressBar.jsx
@@ -7,21 +7,26 @@ import SimpleTooltip from '../shared/SimpleTooltip';
const ProgressBar = ({ magnitude, regression, color }) => (
-
-
- Alert Name
-
-
-
- Detail Hint
- Magnitude
+
+
{filteredAlerts.map(alert => (
diff --git a/ui/perfherder/alerts/AlertTableRow.jsx b/ui/perfherder/alerts/AlertTableRow.jsx
index 2f9d817f374..48fcaeb966f 100644
--- a/ui/perfherder/alerts/AlertTableRow.jsx
+++ b/ui/perfherder/alerts/AlertTableRow.jsx
@@ -2,7 +2,7 @@
import React from 'react';
import PropTypes from 'prop-types';
-import { FormGroup, Input, Label } from 'reactstrap';
+import { Button, FormGroup, Input, Label } from 'reactstrap';
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
import {
faStar as faStarSolid,
@@ -208,13 +208,9 @@ export default class AlertTableRow extends React.Component {
>
+
+ Test and platform
+ Previous Value
+
+ New Value
+ Absolute Difference
+ Magnitude of Change
Confidence
-
-
+
{alertStatus !== 'untriaged' ? (
diff --git a/ui/perfherder/compare/CompareTable.jsx b/ui/perfherder/compare/CompareTable.jsx
index d492336bdef..c5346c3cb57 100644
--- a/ui/perfherder/compare/CompareTable.jsx
+++ b/ui/perfherder/compare/CompareTable.jsx
@@ -51,7 +51,8 @@ export default class CompareTable extends React.PureComponent {
this.header,
)
}
- title={`Permalink to this test table: ${testName}`}
+ title="Permalink to this test table"
+ aria-label={`Permalink to test ${testName} table`}
>
New
Delta
- {/* empty for progress bars (magnitude of difference) */}
Magnitude of Difference
Confidence
@@ -87,152 +87,10 @@ export default class CompareTable extends React.PureComponent {
{data.map(rowLevelResults => (
- {rowLevelResults.name}
-
- {onPermalinkClick && (
-
-
-
- )}
- {rowLevelResults.links &&
- rowLevelResults.links.map(link => (
-
- {` ${link.title}`}
-
- ))}
-
-
-
- {rowLevelResults.originalValue < rowLevelResults.newValue && (
- <
- )}
- {rowLevelResults.originalValue > rowLevelResults.newValue && (
- >
- )}
-
-
- {rowLevelResults.delta &&
- Math.abs(displayNumber(rowLevelResults.deltaPercentage)) !==
- 0 ? (
-
-
- {rowLevelResults.delta ? (
-
-
- {rowLevelResults.delta &&
- rowLevelResults.confidence &&
- rowLevelResults.confidenceText ? (
-
-
- {!hasSubtests &&
- !rowLevelResults.isNoiseMetric &&
- (rowLevelResults.newRetriggerableJobId || !isBaseAggregate) &&
- user.isLoggedIn && (
-
- )}
- {rowLevelResults.originalRuns && (
-
-
->>>>>>> Bug 1600752 - Compare View Changes
))}
diff --git a/ui/perfherder/compare/CompareTableRow.jsx b/ui/perfherder/compare/CompareTableRow.jsx
index 66e6a10a192..1480d94af5e 100644
--- a/ui/perfherder/compare/CompareTableRow.jsx
+++ b/ui/perfherder/compare/CompareTableRow.jsx
@@ -77,6 +77,7 @@ export default class CompareTableRow extends React.PureComponent {
)
}
title="Permalink to this test"
+ aria-label={`Permalink to test ${rowLevelResults.name}`}
>
this.setState({ showMoreResults: !showMoreResults })} > {`show ${showMoreResults ? 'less' : 'more'}`} -
+ )} );