Skip to content

Add custom parser and more features #2

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

philmottin
Copy link

The opened enhancement pull request has bugs on index values on parser function.
I started to fix it and really liked this plug in, so I made some more features.

If you merge, please note that I updated the readme links to my github (still mention your project), so watch out for that.

Features:

  • toggle table header, row numbers
  • set initial row number value
  • set table width
  • custom parser function
  • toggle parse headers, row numbers
  • choose columns to parse
  • added title and footer
  • debug parameter, show all indexes on all cells

Bug fix

  • selector with no class.

Improve performance

  • only call parseTableData() once
  • Removed getHighestColumnCount () and getObjectLength() functions. Using Object.keys(data).length property.

Bug fix
- selector with no class.

Improve performance
- only call parseTableData() once
- Removed getHighestColumnCount () and getObjectLength() functions. Using Object.keys(data).length property.

Features:
- toggle table header, row numbers
- set initial row number value
- set table width
- custom parser function
- toggle parse headers, row numbers
- choose columns to parse
- added title and footer
- debug parameter, show all indexes on all cells
- Close tr tag
- isHead flag inside parser function was wrong
fix error when showTableRowNumber is true and no parser is present. missing value.
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