-
Notifications
You must be signed in to change notification settings - Fork 5
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
(#1589) Updates usa-list twig and styles, storybook structure #1620
Conversation
Viewing Information |
b946b3b
to
7e2275a
Compare
06ea914
to
8060754
Compare
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.
Looks good to me!
{% endif %} | ||
{% set classes = class ~ class1 ~ class2 %} | ||
|
||
{% if ordered %} |
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.
clean up filter and indents:
{% set class = "usa-list" %}
{% if "twoColumn" in variant %}
{% set class = class + " usa-list--two-column" %}
{% endif %}
{% if "unstyled" in variant %}
{% set class = class + " usa-list--unstyled" %}
{% endif %}
{% if ordered %}
<ol class="{{ class }}">
{% else %}
<ul class="{{ class }}">
{% endif %}
{% for item in items %}
<li>{{ item }}</li>
{% endfor %}
{% if ordered %}
</ol>
{% else %}
</ul>
{% endif %}
|
||
const Template = (args) => Component(args); | ||
|
||
export const USWDSList = () => <TestCase css={css} html={Template.bind({})()} />; |
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.
We don't need a USWDS test case. That's really only for USWDS HTML with NCIDS JS. Remove this and the scenarios.
import css from './index.scss'; | ||
|
||
export default { | ||
title: 'Components/usa-list/Default', |
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.
Rename to List
import css from './index.scss'; | ||
|
||
export default { | ||
title: 'Components/usa-list/Modifiers', |
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.
Rename to List
04d1cce
to
0496e32
Compare
0496e32
to
c5d1a85
Compare
Closes #1589