Safari - Links in Tables require multiple clicks or one long click

[mathieu-lessard]

🐛 Bug Report

When a link is in a table, on Safari it requires two clicks if it's the first link in the cell, and rapid clicking if it's not the first link.

The links seem to function normally if you hold the click down and then release.

🤔 Expected Behavior

Regular clicking of the link opens it. This is behaving correctly in Edge, Chrome, Firefox.

😯 Current Behavior

If it's the first link, the cell first is focused, and then the link is clicked.

If it's the second link, the cell is focused on every click unless you click rapidly enough, then the link opens.

For either link, if you hold the click down and then release, the links function as expected.

💁 Possible Solution

Not sure. Seems focus related.

🔦 Context

For Safari users there is some confusion and mildly annoyance to click twice or differently then normal.

💻 Code Sample

https://codesandbox.io/s/react-spectrum-table-safari-double-click-957o02?file=/src/App.js

🌍 Your Environment

Software	Version(s)
react-spectrum	3.20, 3.24.1
Browser	Safari - Version 16.1 (17614.2.9.1.13, 17614)
Operating System	macOS 12.6.1

🧢 Your Company/Team

N/A

🕷 Tracking Issue (optional)

N/A

[snowystinger]

Thanks for the sandbox. I noticed we have a story with a link that seems unaffected https://github.com/adobe/react-spectrum/blob/main/packages/%40react-spectrum/table/stories/Table.stories.tsx#L1929

Which you can try here: https://reactspectrum.blob.core.windows.net/reactspectrum/f7e68cc6ba8c4771837858946995bfa88dc9e9f6/storybook/index.html?path=/story/tableview--table-with-breadcrumb-navigation&providerSwitcher-express=false

I did notice that we're handling the link's onPress ourselves, so that may be a good workaround for you.

As for why it's doing this, I suspect that our grid navigation is using cell first navigation.

[prabirshrestha]

I'm facing the same issue when running in iOS Safari. Having onPress that navigates does seem to be a workaround but it does feel like a hack.