Skip to content

Apply minimal changes/tweaks to restore a responsive narrow view. #15

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

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

albu-diku
Copy link
Contributor

Playing with disabling the custom CSS we apply lead to discovering that the theme we use already had support for a collapsed sidebar view based on a media query that came into effect at 768px.

Arrange applying essentially the same styles at our existing break point of 1100px, remove a couple of conflicting things in our custom CSS and set a handful of basic styles (in line with choices already made in the page) such that the responsive view comes into a working order.

…view.

Playing with disabling the custom CSS we apply lead to discovering that
the theme we use already had support for a collapsed sidebar view based
on a media query that came into effect at 768px.

Arrange applying essentially the same styles at our existing break point
of 1100px, remove a couple of conflicting things in our custom CSS and set
a handful of basic styles (in line with choices already made in the page)
such that the responsive view comes into a working order.
@SebastianPrehn SebastianPrehn requested review from SebastianPrehn and aputtu and removed request for aputtu and SebastianPrehn July 2, 2024 16:11
Copy link
Contributor

@SebastianPrehn SebastianPrehn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, and works when tested against locally.

@SebastianPrehn SebastianPrehn merged commit fe6cfb0 into main Jul 4, 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.

2 participants