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(#94) - Create timeline component #96
Conversation
This pull request introduces 1 alert when merging 0ee344b into d74b113 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging a006f1e into 22da5a8 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 0f0caf9 into 22da5a8 - view on LGTM.com new alerts:
|
fix(1-Platform#94) - Added stepper style APIs and event emitter fix(1-Platform#94) - Added line breaks
packages/opc-timeline/README.md
Outdated
- Sets the active state to the given number | ||
|
||
### variant | ||
- Input: The following strings |
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.
remove this line. start from bullets for variant i.e default and compact. add sub-bullet to explain with text.
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.
Makes sense and made the changes
if (this.variant === 'compact') { | ||
return { | ||
left: html` | ||
<span class="timeline__arrow left" @click="${() => {this._scrollHandler('left')}}"> |
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 shouldn't name classes like this class="timeline__arrow left"
it should be class="timeline__arrow timeline__arrow--left"
or class="timeline__arrow timeline__arrow--right"
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.
Made the changes
Resolves #94
Explain the feature/fix
Added basic for timeline component
Does this PR introduce a breaking change
No
Screenshots
Compact View Screenshot
Ready-for-merge Checklist
If it is a new component
Browsers you have tested in