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

Fix overflow on both mobile and desktop #1745

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

peregrineshahin
Copy link
Contributor

mobile version looks unreadable now, do an auto fix.

@ppigazzini ppigazzini added bug server server side changes gui labels Jul 12, 2023
@ppigazzini
Copy link
Collaborator

ppigazzini commented Jul 12, 2023

LG on DEV the mobile version (PROD, DEV with the old GUI, DEV with the PR)

image

@dav1312
Copy link
Contributor

dav1312 commented Jul 12, 2023

Note that .table-responsive already adds overflow-x: auto;
image

@ppigazzini
Copy link
Collaborator

ppigazzini commented Jul 12, 2023

LG on DEV the desktop version too (PROD, DEV with the old GUI, DEV with the PR).

image

mobile version looks unreadable now, do a scroll fix.
@peregrineshahin
Copy link
Contributor Author

Note that .table-responsive already adds overflow-x: auto; image

Yeah makes sense. pushed

@ppigazzini
Copy link
Collaborator

DEV updated, LGTM

@ppigazzini ppigazzini merged commit 5d74161 into official-stockfish:master Jul 12, 2023
17 checks passed
@ppigazzini
Copy link
Collaborator

PROD updated, thank you @peregrineshahin :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug gui server server side changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants