fix(packages/core): cell onclick should be on inner

Fixes #3518

Author: starpit

packages/core/src/webapp/views/table.ts

@@ -411,8 +411,8 @@ export const formatOneRowResult = (tab: Tab, options: RowFormatOptions = {}) =>
     parent.appendChild(cell)
 
     if (onclick) {
-      cell.classList.add('clickable')
-      cell.onclick = async (evt: MouseEvent) => {
+      inner.classList.add('clickable')
+      inner.onclick = async (evt: MouseEvent) => {
         evt.stopPropagation() // don't trickle up to the row click handler
         if (isPopup() || options.usePip) {
           const { drilldown } = await import('../picture-in-picture')

plugins/plugin-core-support/src/test/core-support/theme.ts

@@ -96,7 +96,7 @@ const clickOnThemeButtonThenClickOnTheme = (clickOn: Theme) => (ctx: Common.ISui
       await new Promise(resolve => setTimeout(resolve, 300))
 
       const checkMarkCell = `${Selectors.SIDECAR} .entity.theme[data-name="${clickOn.name}"] .entity-name.clickable`
-      const nameCell = `${Selectors.SIDECAR} .entity.theme[data-name="${clickOn.name}"] > tr > .clickable`
+      const nameCell = `${Selectors.SIDECAR} .entity.theme[data-name="${clickOn.name}"] > tr > td:not(.not-a-name) > .clickable`
 
       console.error('A', checkMarkCell)
       await ctx.app.client.waitForVisible(checkMarkCell)