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

Odt parser tabs and multiple spaces #79

Merged
merged 6 commits into from Nov 22, 2014

Conversation

Projects
None yet
3 participants
@evfredericksen
Contributor

evfredericksen commented Oct 6, 2014

The current odt parser doesn't look at text:tab (tabs) or text:s (2+ spaces) elements, which results in these characters being ignored in the output. These changes takes these whitespace elements into account, albeit at the cost of some code simplicity. The parser also now looks at text from headings in addition to paragraphs.

@coveralls

This comment has been minimized.

coveralls commented Oct 6, 2014

Coverage Status

Coverage increased (+0.43%) when pulling b438016 on evfredericksen:odt-tabs into 6d5e148 on deanmalmgren:master.

deanmalmgren added a commit that referenced this pull request Nov 22, 2014

Merge pull request #79 from evfredericksen/odt-tabs
Odt parser tabs and multiple spaces

@deanmalmgren deanmalmgren merged commit fa65746 into deanmalmgren:master Nov 22, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@deanmalmgren

This comment has been minimized.

Owner

deanmalmgren commented Nov 22, 2014

I'm terribly sorry it has taken me so long to get this merged. Thanks for the contribution and looking into these whitespace issues more carefully!

deanmalmgren added a commit that referenced this pull request Jan 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment