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

Script inserts missing thead and may move first tr #112

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

Conversation

joltmans
Copy link

This script works just the same as before by default but now if there is
no thead element, a thead is inserted and the first row found is moved into the
thead element. This allows the script to work in places where a proper
thead may not be generated.

This script works just the same as before by default but now if there is
no thead element a thead is inserted and the first row is moved into the
thead element. This allows the script to work in  places where a proper
thead may not be generated.
@joltmans
Copy link
Author

I made this because I was trying to get this to work in Confluence. Confluence does not actually have a thead element in it's rendering of the wiki. This fix really helped solve the issue and I think it's a simple change. It may not play nice with some of the other outstanding merge requests (like multiple header rows) so let me know if you want me to rebase off of any of those first..

Thank you!

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.

1 participant