Hover effects on touch devices

[saskliutas]

We received feedback from consumer using our tree component built on top of iTwinUI Tree regarding hover effects on touch devices: iTwin/viewer-components-react#1428

1. I was wondering if those hover effect on checkbox on touch devices are intended and if not I think it should be disabled in iTwinUI rather then we hacking it out.

2. Regarding tooltips on touch devices I would like to get general answer if they should be shown or not. On desktop I feel that they are useful provide some additional information to user. However, on touch devices I am not sure how they are activated(shown)/deactivated and from the recording in linked issue it seems that that are more annoying than helpful. If that's the case should it also be handled by iTwinUI or we need to have code checking for touch devices and handled that on our end?

[mayank99]

For 1 (hover effects), there is this issue: #1827. We recently fixed it in Tree, but had to hack around our tests (#2618 (comment)). We'd probably need to rework our testing architecture before we can make this across the board. Since your issue happens to be in Tree as well, you probably just need to update iTwinUI versions.

For 2 (tooltip), ideally we show tooltips on touch devices when long pressing. We don't want to introduce an unnecessary disparity between touch and pointer devices (the point about being "more annoying than helpful" has nothing to do with touch devices). Ideally this enhancement should be handled in iTwinUI, but we currently don't have the bandwidth so realistically this would only happen in StrataKit.

[saskliutas]

1. Hover effect happens on Checkbox shown in the tree. I glanced over PR you linked and it looks like it only removes hover on the tree node itself.

[grigasp]

Thanks @mayank99. On your second point - I agree regarding how it should work ideally, but we need to do something about that in the context of iTwinUI. What would be your recommendation there? Hide the tooltip on mobile devices? Where that should be implemented - iTwinUI or our code?

[mayank99]

Re 1: you're right, it's on Checkbox but imo that effect is barely noticeable because of the small target area and the low opacity. I would consider it to be acceptable.

Re 2: I agree something needs to be done, especially for the flickering. Like I said, we don't want to introduce a disparity between touch and pointer devices. Any solution in iTwinUI must include proper support for touch devices. So for the time being, I would suggest disabling the tooltip on mobile devices in your code. Additionally: if the tooltip does not contain any important information and is not used for labeling purposes, then consider removing it altogether or shortening the text.