-
Notifications
You must be signed in to change notification settings - Fork 60
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
docs: update Time component documentation #1959
Conversation
✔️ Deploy preview for fundamental-styles ready! 🔨 Explore the source changes: 0397051 🔍 Inspect the deploy logs: https://app.netlify.com/sites/fundamental-styles/deploys/5fd90a84ddf93d00082843ef 😎 Browse the preview: https://deploy-preview-1959--fundamental-styles.netlify.app |
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 added 2 comments, but the rest looks great for me :)
stories/time/time.stories.js
Outdated
'Default cozy mode implemented mostly to be used mobile devices. There should be at most 1 active wrapper, marked with <code class="docs-code">fd-time__wrapper--active</code> modifier. In those examples it\'s hour or meridian. The rest of items should be marked as collapsed <code class="docs-code">fd-time__item--collapsed</code>. Also meridian (AM/PM) wrapper has to contain <code class="docs-code">fd-time__wrapper--meridian</code> modifier.' | ||
storyDescription: `The default time component is displayed in cozy mode, which is ideal for mobile screens. | ||
|
||
Only one item and wrapper should be active at a time with the \`fd-timewrapper--active\` modifier class. The rest of the items must be collapsed with the \`fd-timeitem--collapsed\` modifier class, displaying only the collapsed slider item (blue) next to the active item. |
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.
Could you change fd-timewrapper--active
to fd-time__wrapper--active
?
stories/time/time.stories.js
Outdated
|
||
Only one item and wrapper should be active at a time with the \`fd-timewrapper--active\` modifier class. The rest of the items must be collapsed with the \`fd-timeitem--collapsed\` modifier class, displaying only the collapsed slider item (blue) next to the active item. | ||
|
||
**Note:** The meridian (AM/PM) wrapper must contain the \`fd-timewrapper--meridian\` modifier class. |
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.
Could you change fd-timewrapper--meridian
to fd-time__wrapper--meridian
?
…AP/fundamental-styles into docs/update-time-documentation
Description
{{ Updated Time docs. }}