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

Add RTF_USE_PNG for rtf to use PNG even DOT_IMAGE_FORMAT is not PNG #10293

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

zeerd
Copy link

@zeerd zeerd commented Sep 8, 2023

Microsoft Word could not include SVG for RTF document.

So, I add a new TAG named RTF_USE_PNG(NO by default) to make doxygen generate PNG for rtf even DOT_IMAGE_FORMAT is not PNG.

For #10292

@albert-github
Copy link
Collaborator

The problem does not only occur with plantuml images but also with other imahes that are generated in svg format (e.g. call / caller graphs) and as far as I can tell they are not included here.
I think it would be better to see whether there is a way to include svg images natively into rtf documents.
Note that when looking with OpenOffice I do see the svg image (I just tested it with a call graph)

@zeerd
Copy link
Author

zeerd commented Sep 8, 2023

I am using the Microsoft Word(2016) which could not include svg.
But, OpenOffice is a good idea that I missed.

I have no OpenOffice and tried libreOffice(6.4.7.2) which could not include svg, too.

And I find that Microsoft Word would zoom large png pictures out to suit the page, but libreOffice will not.

Seems there is not an easy solution for anything, sigh .

@zeerd zeerd changed the title force to use png for rtf when DOT_IMAGE_FORMAT is svg Add RTF_USE_PNG for rtf to use PNG even DOT_IMAGE_FORMAT is not PNG Sep 14, 2023
@zeerd
Copy link
Author

zeerd commented Sep 14, 2023

I am not sure how to make call/caller graphs, but inherit_graph and coll_graph for c++ could swap between png and svg by the new tag now.

delete description for plantuml.
@sonarcloud
Copy link

sonarcloud bot commented Oct 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants