-
Notifications
You must be signed in to change notification settings - Fork 335
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 govuk-equilateral-height function usage in shape-arrow helper #766
Conversation
d1db3d1
to
fea49fe
Compare
Argh, good catch - thanks for that. I think I understand why, but it's pretty annoying that Sass silently outputs the function call verbatim. I wonder if there's a way we could try and catch that if it happens. |
i guess only if we wrap it in a custom mixin that check if function exist using sass's built-in alternatively, if we use |
349e298
to
1e14b05
Compare
Added a test that greps the CSS for anything matching |
1e14b05
to
0817fb9
Compare
fixed the extra semicolon in the test |
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.
👍 Tested with the details component and mobile navigation menu button.
0817fb9
to
20877c0
Compare
Added a big ole' comment around what the test is looking for |
20877c0
to
b684f5a
Compare
In #762 we changed the name of the
govuk-equilateral-height
function to_govuk-equilateral-height
to signify that is is a private function.Unfortunately we forgot to update the name where it was used to create the open/closed indicator for the details component.
This PR fixes this.
Before:
After: