Noissue - Fix broken layout from changeset 1940351cdb4c

Noissue - Fix broken layout from changeset 1940351cdb4c

[ire, 2018-05-30 10:20:55]

Ready for review

What I did was first re-apply Jael's patch then made some changes on top of
that, so the only changes I actually made here were 

- Adding the space between the `html[dir]` and the `.outer` on line 56&61
https://codereview.adblockplus.org/29794555/diff/29794556/static/scss/layout/...
- Re-added the .site-title styles that were previously removed here
https://codereview.adblockplus.org/29794555/diff/29794556/static/scss/layout/...

If you prefer I can create 2 separate reviews so it's more clear which changes I
made vs which changes are from Yael's original commit.

[juliandoucette, 2018-05-30 12:08:57]

On 2018/05/30 10:20:55, ire wrote:
> Ready for review
> 
> What I did was first re-apply Jael's patch then made some changes on top of
> that, so the only changes I actually made here were 
> 
> - Adding the space between the `html[dir]` and the `.outer` on line 56&61
>
https://codereview.adblockplus.org/29794555/diff/29794556/static/scss/layout/...
> - Re-added the .site-title styles that were previously removed here
>
https://codereview.adblockplus.org/29794555/diff/29794556/static/scss/layout/...
> 
> If you prefer I can create 2 separate reviews so it's more clear which changes
I
> made vs which changes are from Yael's original commit.

LGTM

I'd prefer if you pushed this in two separate commits (one by Yael, another by
you) and mentioned that your commit fixes hers in the commit messages.

[ire, 2018-05-30 12:53:55]

On 2018/05/30 12:08:57, juliandoucette wrote:
> On 2018/05/30 10:20:55, ire wrote:
> > Ready for review
> > 
> > What I did was first re-apply Jael's patch then made some changes on top of
> > that, so the only changes I actually made here were 
> > 
> > - Adding the space between the `html[dir]` and the `.outer` on line 56&61
> >
>
https://codereview.adblockplus.org/29794555/diff/29794556/static/scss/layout/...
> > - Re-added the .site-title styles that were previously removed here
> >
>
https://codereview.adblockplus.org/29794555/diff/29794556/static/scss/layout/...
> > 
> > If you prefer I can create 2 separate reviews so it's more clear which
changes
> I
> > made vs which changes are from Yael's original commit.
> 
> LGTM
> 
> I'd prefer if you pushed this in two separate commits (one by Yael, another by
> you) and mentioned that your commit fixes hers in the commit messages.

Yup that was the plan. Thanks!

[ire, 2018-05-30 12:57:53]

Commits:

- [Noissue - Fix broken layout from changeset
1940351cdb4c](https://hg.adblockplus.org/web.acceptableads.com/rev/19f4e3949577)
(Ire Aderinokun)
- [Fixes #6 - Unfix primary
navigation](https://hg.adblockplus.org/web.acceptableads.com/rev/d2d8b369d2e9)
(Yael Stein)