Skip to content

Conversation

marbemac
Copy link
Contributor

@marbemac marbemac commented Dec 3, 2021

To help visually connect prop to extra values on right if there are any.

2021-12-03 15 54 08

@marbemac marbemac requested review from a team and Nezteb December 3, 2021 20:52
@billiegoose
Copy link
Contributor

screenshot looks good! gonna try running storybook locally. I'm curious how it handles on touch screens or with keyboard focus.

@marbemac
Copy link
Contributor Author

marbemac commented Dec 3, 2021

Good call! I'll check too

@marbemac
Copy link
Contributor Author

marbemac commented Dec 3, 2021

So obv no hover on touch screen, but will show on tap in this case 🤷‍♂️

2021-12-03 16 08 05

@marbemac
Copy link
Contributor Author

marbemac commented Dec 3, 2021

On the other side of things, I have a whole lot of ideas re keyboard accessibility / navigation that I'd like to visit at some point.

@billiegoose
Copy link
Contributor

(Keyboard accessibility is a big ask obviously, but touch support is simple enough.)

Edit: Looks like setHoveredNode is called when nodes are touched, so looks nice on mobile 👍🏻

@marbemac marbemac enabled auto-merge (squash) December 3, 2021 21:13
Copy link
Contributor

@billiegoose billiegoose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks beautiful!

@marbemac marbemac merged commit 20e90ea into master Dec 3, 2021
@marbemac marbemac deleted the mbm/feat/hover-guidelines branch December 3, 2021 21:14
@stoplight-bot
Copy link
Collaborator

🎉 This PR is included in version 4.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants