-
Notifications
You must be signed in to change notification settings - Fork 716
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
Version 3.0 ToDos #222
Comments
Re: tests, Puppeteer looks promising. Decent tutorial on using it with Mocha: https://medium.com/@ankit_m/ui-testing-with-puppeteer-and-mocha-part-1-getting-started-b141b2f9e21 |
|
Great work on this @jedfoster. The general functionality works, but the use of document.querySelector causes a few problems... The moreLink, for example, doesn't get replaced with lessLink on toggle. Have you had any thoughts on supporting this being used within shadowDom? |
@olivercastle No, I hadn't even thought of using this in a web component, but that's a fantastic idea. Unfortunately, I have zero experience with that technology, and don't have any idea how to start. Pull requests are always welcome. |
https://github.com/jedfoster/Readmore.js/tree/version-3.0
<script src="/node_modules/readmore-js/dist/readmore.js"></script>
)-moz
, et al?)destroy()
methodtoggle()
andconstructor()
to work withquerySelectorAll
master
branchRefactorNot sure of the utility of this;toggle()
to expose as a static methodtoggle
depends onoptions
which is an instance property.The text was updated successfully, but these errors were encountered: