-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
"View the solution" and "Skip this move" are too close together on mobile #15322
Comments
I created a pull request for this fix: #15324 |
I played around with it a bit (without being a UII designer), but if you add the "button" class to the anchors in choices and you use this:
|
@Siderite You're right. Buttons look even better. Could you please make this change? |
I am not a developer for Lichess
…On Sat, May 18, 2024, 20:37 Nikhilesh Jasuja ***@***.***> wrote:
@Siderite <https://github.com/Siderite> You're right. Buttons look even
better. Could you please make this change?
—
Reply to this email directly, view it on GitHub
<#15322 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAB7TMJID4OMILS5HUIRRDLZC6GUDAVCNFSM6AAAAABH4PGOTGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJYHA4TMMJWGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I use my phone a lot to play (website, not the app). In the analysis/retro section the links for "View the solution" and "Skip this move" are too close so I end up clicking "Skip this move" instead of "View the solution".
The fix for this is quite simple. Add the following CSS in ui/analyse/css/_retro.scss inside the
.retro-box
block:The text was updated successfully, but these errors were encountered: