Skip to content
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

Added pageinate-links custom tag support & improved compatibility #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chazeon
Copy link

@chazeon chazeon commented Jul 16, 2017

  • Allows <paginate-links> custom tag (as is mentioned in issue Pagination links not displaying correctly with Semantic-UI #46) to support a wider range of UI framework
  • Click handler could be placed at various layers
  • Updated README.md with these introductions of this feature
  • Fixed browser compatibility: many browsers (such as Edge) does not support forEach() function on NodeList (return value of querySelectAll()).

* Allows `<paginate-links>` custom tag
* Click handler could be placed at various layers
* Updated README with these features
* Fixed browser compatibility: many browsers (such as Edge) does not
  support `forEach()` function on `NodeList` (return value of
  `querySelectAll()`).
@chasegiunta
Copy link

Ah, just ran into this issue with forEach() on NodeList. Was trying to use polyfill.io but even they don't support it since it's not actually in spec. Would be great if this can be merged in. Thanks for the work @chazeon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants