fix: agenda tooltip with keyboard focus

[gavinbarron]

closes #2236

PR Type

• Bugfix

Description of the changes

add a tab stop to titles in agenda events with tooltip

PR checklist

• Project builds (yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)
• All public APIs (classes, methods, etc) have been documented following the jsdoc syntax
• Stories have been added and existing stories have been tested
• Added appropriate documentation. Docs PR:
• Accessibility tested and approved
• License header has been added to all new source files (yarn setLicense)
• Contains NO breaking changes

Other information

[microsoft-github-policy-service]

Thank you for creating a Pull Request @gavinbarron.

This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:

• I have verified a documentation PR has been linked and is approved (or not applicable)
• I have ran this PR locally and have tested the fix/feature
• I have verified that stories have been added to storybook (or not applicable)
• I have tested existing stories in storybook to verify no regression has occured
• I have tested the solution in at least two browsers (Edge + 1 non-Chromium based browser)

[github-actions]

The updated storybook is available here

[gavinbarron]

@vagpt could you please test this fix

[vagpt]

Hi @gavinbarron,

This issue is partially fixed as in current Tooltip is appearing, but it appears when keyboard focus is landing on non-interactive element i.e. 'Lunch or Friday Unwinder'. So it is not properly fixed as both are non-interactive elements.

So as per the expected Tooltip should be as soon as keyboard focus entirely lands on the 12:30 AM- 1:30 AM section.

Attachment:

MGTP.-.Partially.fixed.mp4

[gavinbarron]

@vagpt as outlined in the related issue we cannot trigger the tooltip based on focus of the card, it's simply NOT acheivable with the set of components being used

At this point I propose we remove the tooltip completely @sebastienlevert, thoughts?

[sebastienlevert]

Agree @gavinbarron. Let's remove it as anyways it conveys partial information.

[gavinbarron]

@vagpt we've removed the tooltip as there is no solution that looked good for what was a low quality feature

[github-actions]

The updated storybook is available here

[vagpt]

@gavinbarron and @sebastienlevert

Now, there is no tooltip appearing as per your above comment so we are good to close this issue.

[github-actions]

The updated storybook is available here