Skip to content

Add default font color for Word (.docx) #2700

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 33 commits into from
Jan 26, 2025
Merged

Conversation

Collie-IT
Copy link
Contributor

@Collie-IT Collie-IT commented Nov 21, 2024

Adds the abillity to add a default font color for generated .docx.

Description

Adds the abillity to add a default font color for generated .docx.

Fixes # (issue)

Checklist:

  • My CI is 🟢
  • I have covered by unit tests my new code (check build/coverage for coverage report)
  • I have updated the documentation to describe the changes
  • I have updated the changelog

Adds the abillity to add a default font color for generated .docx.
@coveralls
Copy link

coveralls commented Nov 21, 2024

Coverage Status

coverage: 96.916% (-0.005%) from 96.921%
when pulling 58d5017 on Collie-IT:master
into 10ae499 on PHPOffice:master.

Collie-IT and others added 12 commits November 24, 2024 21:34
As suggested

Co-authored-by: Progi1984 <progi1984@gmail.com>
As suggested

Co-authored-by: Progi1984 <progi1984@gmail.com>
Add default font color
Add documentation default font color
Correct default value
As suggested

Co-authored-by: Progi1984 <progi1984@gmail.com>
As suggested

Co-authored-by: Progi1984 <progi1984@gmail.com>
@Progi1984 Progi1984 self-assigned this Jan 11, 2025
Copy link
Member

@Progi1984 Progi1984 left a comment

Choose a reason for hiding this comment

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

One improvement in unit test and I can merge it after.

Good job @Collie-IT

@Progi1984 Progi1984 added this to the 1.4.0 milestone Jan 14, 2025
@Progi1984
Copy link
Member

@Collie-IT It seems there is a merge conflict. Could you fix it ?

@Progi1984 Progi1984 merged commit f1470d0 into PHPOffice:master Jan 26, 2025
30 checks passed
@Progi1984
Copy link
Member

Thanks @Collie-IT for your contribution 🥳

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants