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

Updates sidebar navigation style to be much tighter. #6

Merged
merged 2 commits into from
Nov 28, 2016

Conversation

marineb
Copy link
Contributor

@marineb marineb commented Nov 22, 2016

No description provided.

@marineb marineb force-pushed the mboudeau/WE-6522-nav-and-homepage-tweaks branch from 2c07ac9 to 8b5e79a Compare November 23, 2016 00:46
@@ -82,7 +82,7 @@
<div class="scrollable-body">
<div class="{{if media.isLargeAndUp 'box box--nopad'}}">

<ul class="list list--noborder flush text--spacing text--semi {{unless media.isLargeAndUp 'list--inline list--spaced text--small' 'alignright text--medium '}}">
<ul class="list list--noborder flush {{unless media.isLargeAndUp 'list--inline list--spaced text--small' 'alignright '}}">
Copy link
Contributor

Choose a reason for hiding this comment

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

can you get rid of the trailing whitespace after 'alignright?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, it was there before, so I had left it in there. I should have checked if it was there for a reason. Done!

@marineb marineb merged commit f66a22f into master Nov 28, 2016
@marineb marineb deleted the mboudeau/WE-6522-nav-and-homepage-tweaks branch November 28, 2016 18:53
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