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
Add translation nav component #289
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of minor comments!
margin-bottom: $gutter; | ||
border-bottom: 1px solid $border-colour; | ||
@include responsive-top-margin; | ||
@include core-16; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know this is just a straight copy from government-frontend, but makes sense to tidy up while we're here - in this case, could use your suggestion of always putting @includes
at the top of a CSS block
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good shout.
@@ -0,0 +1,72 @@ | |||
require "rails_helper" | |||
|
|||
describe "Search", type: :view do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a copy paste error?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
D'oooooooh..
7ab5036
to
f34540a
Compare
f34540a
to
5897c89
Compare
5897c89
to
cd69eec
Compare
Moving from government-frontend to the gem. A separate PR will be raised to remove it from government-frontend. No changes to the component other than changing the CSS namespace from app-c to gem-c.
Trello cards: