himanish-star commented on code in PR #15400:
URL: https://github.com/apache/pinot/pull/15400#discussion_r2018979032
##########
pinot-controller/src/main/resources/app/components/Table.tsx:
##########
@@ -562,7 +564,7 @@ export default function CustomizedTables({
) : (
<StyledTableCell
key={idx}
- className={isCellClickable ?
classes.isCellClickable : (isSticky ? classes.isSticky : '')}
+ className={isCellClickable &&
(!makeOnlyFirstCellClickable || !idx) ? classes.isCellClickable : (isSticky ?
classes.isSticky : '')}
Review Comment:
Since `isCellClickable` makes all cells clickable and underlined and that is
what may not be wanted in some scenarios. I've added this new
`makeOnlyFirstCellClickable` which makes sure that only the first cell is
clickable if the flag is set to true
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
