Check if element clicked is link to skip the toggle of the section (#2632)

Author: sroy3

webview/src/plots/components/App.test.tsx

@@ -428,6 +428,32 @@ describe('App', () => {
     })
   })
 
+  it('should not toggle the checkpoint plots section if a link is clicked', () => {
+    renderAppWithOptionalData({
+      checkpoint: checkpointPlotsFixture
+    })
+
+    const checkpointsTooltipToggle = screen.getAllByTestId(
+      'info-tooltip-toggle'
+    )[2]
+    fireEvent.mouseEnter(checkpointsTooltipToggle, {
+      bubbles: true,
+      cancelable: true
+    })
+
+    const tooltip = screen.getByTestId('tooltip-checkpoint-plots')
+    const tooltipLink = within(tooltip).getByRole('link')
+    fireEvent.click(tooltipLink, {
+      bubbles: true,
+      cancelable: true
+    })
+
+    expect(mockPostMessage).not.toHaveBeenCalledWith({
+      payload: { [Section.CHECKPOINT_PLOTS]: true },
+      type: MessageFromWebviewType.TOGGLE_PLOTS_SECTION
+    })
+  })
+
   it('should not toggle the checkpoint plots section when its header is clicked and the content of its tooltip is selected', async () => {
     renderAppWithOptionalData({
       checkpoint: checkpointPlotsFixture

webview/src/plots/components/PlotsContainer.tsx

@@ -18,6 +18,7 @@ import {
   Lines
 } from '../../shared/components/icons'
 import { isSelecting } from '../../util/strings'
+import { EventTargetWithNodeName } from '../../util/objects'
 
 export interface CommonPlotsContainerProps {
   onResize: (size: PlotSize) => void
@@ -139,7 +140,10 @@ export const PlotsContainer: React.FC<PlotsContainerProps> = ({
 
   const toggleSection = (e: MouseEvent) => {
     e.preventDefault()
-    if (!isSelecting([title, SectionDescription[sectionKey].props.children])) {
+    if (
+      !isSelecting([title, SectionDescription[sectionKey].props.children]) &&
+      !['A', 'BUTTON'].includes((e.target as EventTargetWithNodeName).nodeName)
+    ) {
       sendMessage({
         payload: {
           [sectionKey]: !sectionCollapsed

webview/src/util/objects.ts

@@ -15,3 +15,7 @@ export type Obj = { [key: string]: Any }
 
 export const keepReferenceIfEqual = (old: BaseType, recent: BaseType) =>
   isEqual(old, recent) ? old : recent
+
+export interface EventTargetWithNodeName extends EventTarget {
+  nodeName: string
+}