-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hover over theme patterns displays the default cursor instead of a pointer (hand) and there's no outline #128
Comments
Related to #126 |
Ah, I was going to do that.
Good catch on the cursor 👍🏻 The styles for this were only designed for the previewer, so I kind of … backported them to something that makes sense on this page. I agree the border change should be the same though, and since it was just updated in plugins, I'll go ahead and have the hover darken border color on patterns too. |
Good issue. A few things here, let's add a transition on both. Something a la That's a choppy GIF but hopefully the animation comes across. For reduced motion we have this mixin in the block editor:
Steve is right about the border width, there's also something off about the radius: It's because there's an abs-positioned element on top of a pattern preview, and the outer radius is 4px, the inner radius is 3, but it doesn't fully add up. I wonder if the border could be set on .wporg-theme-listbox .wp-block-wporg-screenshot-preview instead? It should be 4px radius, 1px width, 1.5px only on focus. Can have the same easing. |
Moving the issue from #139 to a comment here, it doesn't need to be a separate issue.
|
I'll take your word for it. In some other contexts, we've used inset shadows to accomplish the same. If that gets paired with something like a |
Basically, the hover states for patterns on single theme pages are not the same as the hover states for style variations, and I think they should be the same. Patterns are missing the highlighted border and the pointer cursor style. Unfortunately the screen recording tool I'm using is not picking up the different cursor styles, but you can test by looking at the Twenty Twenty-Four page.
We also might want some transition applied to the animation, but I leave that to @WordPress/meta-design
hover-states.mp4
The text was updated successfully, but these errors were encountered: