Skip to content

Limit the size of content on very large screens #198

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

Closed
wants to merge 1 commit into from
Closed

Limit the size of content on very large screens #198

wants to merge 1 commit into from

Conversation

didaquis
Copy link

I've added some media queries to limit the size of content on very large screens.

I'm not a CSS expert so I would like to get feedback from other users about these changes.

@didaquis
Copy link
Author

@Zearin would you like to give me some feedback?

<img alt="Creative Commons License" class="license-badge" src="images/cc-by-sa.svg"
width="80" height="15" /></a>
<br>
Copyright &#169; 2011-2019 the
Copy link

Choose a reason for hiding this comment

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

© 2011–2020, now ;-)

Copy link
Author

Choose a reason for hiding this comment

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

I'm not sure about that. On my opinion that change will be another topic (pull request) completely different. 😉

@Zearin
Copy link

Zearin commented Jan 22, 2020

(Sorry it took me so long to respond!)

Looks okay to me. If there are further tweaks, they can happen in future Pull Requests. If you’ve tested it on screens that big (which I don’t have at home, so I’ll just trust you), I say it’s worth a merge.

@didaquis
Copy link
Author

If you’ve tested it on screens that big (which I don’t have at home, so I’ll just trust you), I say it’s worth a merge.

Yes I tried on a double monitor (extended desktop).

@didaquis
Copy link
Author

@hegemonic ping! ;-)

@didaquis didaquis closed this May 15, 2024
@didaquis didaquis deleted the ui/limit-content-size branch May 15, 2024 06:25
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