elyra-ai#1981 Fixed unable to see tooltip for table headers having ch…

…eckbox Signed-off-by: Neha Gokhale <[email protected]>
nmgokhale · May 23, 2024 · bcaf785 · bcaf785
1 parent 486bbfa
commit bcaf785
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/.../src/common-properties/components/truncated-content-tooltip/truncated-content-tooltip.jsx b/.../src/common-properties/components/truncated-content-tooltip/truncated-content-tooltip.jsx
@@ -30,7 +30,7 @@ export default class TruncatedContentTooltip extends React.Component {
 			tooltipText = String(this.props.tooltipText);
 		}
 		if (this.props.content && this.props.content.type === Checkbox && this.tooltipRef && this.tooltipRef.firstChild) {
-			truncatedRef = this.tooltipRef.firstChild.lastChild; // checkbox label is in div -> label -> span
+			truncatedRef = this.tooltipRef.firstChild.lastChild.previousSibling; // checkbox label is in div -> label -> span
 		}
 		const tooltip = (
 			<div className="properties-tooltips">

diff --git a/canvas_modules/harness/cypress/e2e/properties/properties-tooltips.cy.js b/canvas_modules/harness/cypress/e2e/properties/properties-tooltips.cy.js
@@ -107,8 +107,7 @@ describe("Test if tips show up in table headers correctly", function() {
 		cy.saveWideFlyout("Configure Rename fields");
 	});
 
-	// TODO : Enable this test after fixing table tooltip issue.
-	it.skip("Test if tips show when checkbox in header has ellipsis", function() {
+	it("Test if tips show when checkbox in header has ellipsis", function() {
 		cy.toggleCategory("More Tables");
 		cy.openSubPanel("Configure Dummy Types");
 		cy.get(".properties-wf-children div[data-id='properties-vt-header-override']")