Skip to content
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

Added padding between "View the solution" and "Skip this move" #15324

Closed
wants to merge 4 commits into from

Conversation

diffen
Copy link

@diffen diffen commented May 17, 2024

Added padding between "View the solution" and "Skip this move" in the game retro section.

In the analysis/retro section, the links for "View the solution" and "Skip this move" are too close so users end up clicking "Skip this move" instead of "View the solution".

This is a fix for Issue-15322 that I reported.

… game retro section.

In the analysis/retro section, the links for "View the solution" and "Skip this move" are too close so users end up clicking "Skip this move" instead of "View the solution".
Added padding between "View the solution" and "Skip this move"
@diffen
Copy link
Author

diffen commented May 20, 2024

There is no reason for linting to fail. How can I get this PR accepted?

@randompearl-stack
Copy link
Contributor

I copied your change to gitpod and ran ./lila-docker format, and Prettier suggested to remove the last empty line (55).

The extra empty line was causing the linting to fail.
@ornicar ornicar closed this in c79e6a0 May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants