Skip to content

Commit

Permalink
Merge pull request #247 from IBMa/joho-removeParentChildTooltips
Browse files Browse the repository at this point in the history
remove Parent Child Tooltips
  • Loading branch information
ErickRenteria authored Oct 5, 2020
2 parents 9bdd4e4 + b3ea265 commit 18835b4
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions accessibility-checker-extension/src/ts/devtools/ReportRow.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

import React, { RefObject } from "react";

import ReactTooltip from "react-tooltip";
// import ReactTooltip from "react-tooltip";

import { IReportItem, valueMap, ICheckpoint, IReport} from "./Report";

Expand Down Expand Up @@ -190,15 +190,16 @@ export default class ReportRow extends React.Component<IReportRowProps, IReportR
return <React.Fragment>
{!this.props.focusedViewFilter || (focusedView && (item.selected || item.selectedChild)) ?
(this.props.dataFromParent[0] || this.props.dataFromParent[1] && val === "Violation" || this.props.dataFromParent[2] && val === "Needs review" || this.props.dataFromParent[3] && val === "Recommendation") ?
// (<div data-tip data-for={item.selected ? "selectedTip" : "selectedChildTip" } tabIndex={0} role="row" style={{cursor:'pointer'}} aria-rowindex={++rowindex} aria-selected={!!item.selected} className={"bx--row itemDetail"+(item.selected ? " selected": "")+(item.selectedChild ? " selectedChild": "")} onClick={this.props.selectItem.bind(this, item, this.props.group.checkpoint)} onKeyDown={this.onKeyDown.bind(this)}>
(<div data-tip data-for={item.selected ? "selectedTip" : "selectedChildTip" } tabIndex={0} role="row" style={{cursor:'pointer'}} aria-rowindex={++rowindex} aria-selected={!!item.selected} className={"bx--row itemDetail"+(item.selected ? " selected": "")+(item.selectedChild ? " selectedChild": "")} onClick={this.props.selectItem.bind(this, item, this.props.group.checkpoint)} onKeyDown={this.onKeyDown.bind(this)}>
{focusedView && item.selected ?
{/* {focusedView && item.selected ?
<ReactTooltip id="selectedTip" place="top" effect="solid">
Parent element issue
</ReactTooltip> : ""}
{focusedView && item.selectedChild ?
<ReactTooltip id="selectedChildTip" place="top" effect="solid">
Child element issue
</ReactTooltip> : ""}
</ReactTooltip> : ""} */}
<div role="cell" className="bx--col-sm-1"> </div>
<div role="cell" className="bx--col-sm-3">
<div className="itemMessage">
Expand Down

0 comments on commit 18835b4

Please sign in to comment.