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

Fix for the formatting issues with bullet points for exported docx plans #2268

Merged
merged 1 commit into from
Nov 4, 2019

Conversation

johnpinto1
Copy link
Contributor

@johnpinto1 johnpinto1 commented Oct 24, 2019

when viewed in an old Word editor.

The fix follows the solution suggested in the HtmlToWord bug "List item
bullets corrupted in Microsoft Word"
karnov/htmltoword#73 suggested by David
Verhasselt @dv.

Changes:

  • Firstly, removed earlier attempted fix in commit
    5ef7e05 (Fix for format issues for
    bullet points in Plan docx exports).
  • Made a copy of Htmltoword (1.1.0) xslt stylesheets functions.xslt
    and numbering.xslt and made changes suggested by List item bullets corrupted in Microsoft Word karnov/htmltoword#73
  • Prepended the Htmltoword library XSLTHelper method xslt_path() in
    config/initializers/htmlto_word_xslt_helper.rb. This method ensures
    the Htmltoword library picks up the changed numbering.xslt template instead of
    the library's version.

Fix for issue #2147.

@dsisu Tested the downloaded docx in an old Word editor.
Test_Bullets_2 (8).docx

Selection_032

Copy link
Contributor

@xsrust xsrust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. We'll have to watch for this being patched on the main HTMLtoword gem.

@briri
Copy link
Contributor

briri commented Nov 1, 2019

@johnpinto1 please pin the version of the gem in the the Gemfile

when viewed in an old Word editor.

The fix follows the solution suggested in  the HtmlToWord bug "List item
bullets corrupted in Microsoft Word"
karnov/htmltoword#73 suggested by David
Verhasselt @dv.

Changes:
   - Firstly, removed earlier attempted fix in commit
     5ef7e05 (Fix for format issues for
     bullet points in Plan docx exports).
   - Made a copy of Htmltoword (1.1.0) xslt stylesheets functions.xslt
     and numbering.xslt and made changes suggested by karnov/htmltoword#73
   - Prepended the Htmltoword library XSLTHelper method xslt_path() in
     config/initializers/htmlto_word_xslt_helper.rb. This method ensures
     the Htmltoword library picks up the changed  numbering.xslt template instead of
     the library's version.
   - pinned version of htmltoword to 1.1.0

Fix for issue #2147.
@johnpinto1
Copy link
Contributor Author

@briri @xsrust Pinned gem version of htmltoword to 1.1.0 and rebased as suggested.

Copy link
Contributor

@briri briri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good @johnpinto1. Thanks for pegging the htmltoword gem version

@briri briri merged commit 26f2cb1 into development Nov 4, 2019
@briri briri deleted the issue2147 branch December 9, 2019 17:06
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.

3 participants