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

style: update CSS to resolve whitespace issue on devices under 704px height. #12405

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mohammednumaan
Copy link

Summary

Fixes #12202

Problem

This PR fixes the white-space issue in the sidebar on devices under 704px height. The issue is most likely due to the section.side.place class which has the following CSS:

.place {
   align-self: center;
   grid-row: 2/3;
   justify-self: center;
   margin-bottom: 0;
}

Here, the grid-row is set to 2/3. The element takes 2 rows of space and causes the large white-space shown below.

Solution

The solution to fix this issue would be to modify the grid-row property in the .place class. By setting it to auto the issue is resolved.


Screenshots

The screenshots are attached below for reference.

Before

image

After

image


@mohammednumaan mohammednumaan requested a review from a team as a code owner January 7, 2025 14:40
@mohammednumaan mohammednumaan changed the title Update CSS to resolve whitespace issue on devices under 704px height. style: update CSS to resolve whitespace issue on devices under 704px height. Jan 7, 2025
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.

CSS: Sidebar appears largely blank at mobile screens without placements
1 participant