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

Background and border color transparency support for textboxes when color is undefined #2555

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

Chudy20007
Copy link
Contributor

@Chudy20007 Chudy20007 commented Jan 18, 2024

Description

In project we use text boxes for imprints and side notes on a word document. It's currently not possible to use non-white background with border. As a simple fix I implement simple check if border or background color is set - if not then text box should be rendered with a transparent colors, else current behavior will be used.

Fixes # (issue)

Checklist:

  • I have run composer run-script check --timeout=0 and no errors were reported
  • The new code is covered by unit tests (check build/coverage for coverage report)
  • I have updated the documentation to describe the changes

@coveralls
Copy link

Coverage Status

coverage: 97.218% (+0.001%) from 97.217%
when pulling 59ed844 on Chudy20007:transparent-textbox
into 2f4da6e on PHPOffice:master.

@mzgs
Copy link

mzgs commented Sep 26, 2024

Thanks a lot. its woking.

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.

@Chudy20007 Could you add some unit tests and update the changelog, please ?

@Progi1984 Progi1984 added the Status: Waiting for feedback Question has been asked, waiting for response from PR author label Feb 12, 2025
@Chudy20007 Chudy20007 requested a review from Progi1984 February 12, 2025 16:26
@Chudy20007 Chudy20007 force-pushed the transparent-textbox branch 2 times, most recently from 3c61539 to e123278 Compare February 12, 2025 16:52
@Progi1984 Progi1984 merged commit 2a524e2 into PHPOffice:master Feb 12, 2025
29 checks passed
@Progi1984 Progi1984 removed the Status: Waiting for feedback Question has been asked, waiting for response from PR author label Feb 12, 2025
@Progi1984 Progi1984 added this to the 1.4.0 milestone Feb 12, 2025
@Progi1984
Copy link
Member

@Chudy20007 Thank you for your contribution 🥳

@coveralls
Copy link

coveralls commented Feb 13, 2025

Coverage Status

coverage: 96.757% (+0.008%) from 96.749%
when pulling 1e5098f on Chudy20007:transparent-textbox
into 66f0a2c on PHPOffice:master.

MichaelPFrey pushed a commit to MichaelPFrey/PHPWord that referenced this pull request Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants