-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Feature] Add a readmore
component
#106
base: develop
Are you sure you want to change the base?
Conversation
Export SizeUnchanged@studiometa/ui
|
Codecov Report
@@ Coverage Diff @@
## develop #106 +/- ##
===========================================
- Coverage 26.64% 26.16% -0.49%
===========================================
Files 46 47 +1
Lines 1141 1162 +21
Branches 211 216 +5
===========================================
Hits 304 304
- Misses 771 792 +21
Partials 66 66
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
This is a nice first draft, well done @Liax!
readmore
component
advanced usage. Implement dedicated methods for the show anb hide
Nice ! |
*/ | ||
hide() { | ||
this.$refs.hiddenContent.classList.add('hidden'); | ||
this.$refs.btn.innerHTML = this.$options.btnLabelMore; |
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.
suggestion: innerHTML
might be unsafe. Consider using textContent
instead 😇
Objective
First draft of a
readmore
component.To do