Skip to content

Commit 8051aae

Browse files
committed
Fix Tableau logical operator expansion
Relax normalizeLogicalOperator to recognize any tag ending in 'Op' as a potential logical operator, not just those with class='logical-operator'. This ensures operators like domainOp, aggregateOp, selectOp are placed in expandedChildren and shown by default rather than collapsedChildren. Fixes rendering of logical-operator-class-domain.xml and similar plans where operators are collapsed by default when they should be expanded.
1 parent c1c1d87 commit 8051aae

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎query-graphs/src/tableau.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ function normalizeLogicalOperator(tag: string, clazz?: string) {
2222
if (tag == "logical-operator") {
2323
return clazz;
2424
}
25-
if (clazz == "logical-operator" && tag.endsWith("Op")) {
26-
return tag.substring(0, tag.length - 2);
25+
if (tag.endsWith("Op")) {
26+
return clazz == "logical-operator" ? tag.substring(0, tag.length - 2) : tag.substring(0, tag.length - 2);
2727
}
2828
return undefined;
2929
}

0 commit comments

Comments
 (0)