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

Dont set background color if none is provided in options #1285

Merged
merged 1 commit into from
Nov 15, 2021

Conversation

erlend-axelsson
Copy link
Contributor

After comparing the output of this library and what is
produced by Microsoft Word, it seems that document
background color is not set by default by Word.

Removing the default #FFFFFF background color enables the
generated documents to be readable in dark mode.

This seems to fix #1093

After comparing the output of this library and what is
produced by Microsoft Word, it seems that document
background color is not set by default by Word.

Removing the default #FFFFFF background color enables the
generated documents to be readable in dark mode.

This seems to fix dolanmiu#1093
@codecov-commenter
Copy link

Codecov Report

Merging #1285 (ba963b0) into master (0757295) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1285   +/-   ##
=======================================
  Coverage   99.30%   99.30%           
=======================================
  Files         314      314           
  Lines        3600     3600           
  Branches      454      454           
=======================================
  Hits         3575     3575           
  Misses         24       24           
  Partials        1        1           
Impacted Files Coverage Δ
...ocument/document-background/document-background.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0757295...ba963b0. Read the comment docs.

@dolanmiu dolanmiu merged commit c28e6ac into dolanmiu:master Nov 15, 2021
@dolanmiu
Copy link
Owner

Perfect. Merged

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.

Word docs created don't work well in Dark Mode by default
3 participants