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 the media queries used for our grid #791

Merged
merged 2 commits into from
Feb 12, 2016

Conversation

graygilmore
Copy link
Contributor

This fixes a bug where content would be cut off. This is a simple stop-gap before we rewrite this properly.

Before:
screen shot 2016-02-02 at 8 27 12 pm

After:
screen shot 2016-02-02 at 8 25 59 pm

@jhawthorn
Copy link
Contributor

👍 Looks good to me

In your opinion should we backport this to the 1.2 branch?

@graygilmore
Copy link
Contributor Author

Good question. Yes, this should have been a part of the left navigation introduction.

@Sinetheta
Copy link
Contributor

Good fix for an unfortunate problem. It's too bad we don't have element queries for this. Maybe a comment? // old width + sidebar width

@jhawthorn
Copy link
Contributor

@graygilmore pointed out to me that this is used by both frontend and backend. We probably want to make this change only on the backend.

@graygilmore
Copy link
Contributor Author

Yes, I will pull this out of core and put a file in each of the backend/frontend. Then I'll be able to use skeleton.css.scss and use the $width-sidebar in the file.

We need to make changes to how the skeleton operates in the backend with
the addition of the left navigation. Unfortunately, the changes required
would apply to both the admin and the frontend so we need to separate
these files.

Ideally, the admin and the frontend wouldn't share any assets at all
which this commit also pushes us in the direction of.
Since we added the left-side navigation our media queries haven't been
accounted for. This commit fixes an issue where content would be off the
page and a horizontal scroll bar would appear.
@graygilmore
Copy link
Contributor Author

@Sinetheta moved the file into Sass-land so I can do 768px + $width-sidebar which should be clear enough.

@jhawthorn added a new commit which addresses the admin/front-end issue: graygilmore@a30a920

@jhawthorn
Copy link
Contributor

👍 again from me

@graygilmore graygilmore mentioned this pull request Feb 5, 2016
@gmacdougall
Copy link
Member

👍

gmacdougall added a commit that referenced this pull request Feb 12, 2016
Fix the media queries used for our grid
@gmacdougall gmacdougall merged commit 5f98955 into solidusio:master Feb 12, 2016
@graygilmore graygilmore deleted the fix_media_queries branch February 12, 2016 20:17
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.

4 participants