BIRT 4.15: SVG images won't be displayed on HTML-preview which are embedded on TEXT-element (type HTML)

[speckyspooky]

BIRT 4.15: SVG images won't be displayed on HTML-preview which are embedded on TEXT-element (type HTML)

[speckyspooky]

Yes, of course with that extension it works. I have done a small change at 2 classes which checks "external" sources
and added the "data:"-URL-protocol key and the first try shows the old behavior where the svg-data-URL will be embedded.

It seems to me that the behavior is changed due to the changed tidy-parser behavior and therefore we got some effects.

I will take a look into the file extension way because there we would need a special role to get the extension from the URL.

[merks]

My sense is that inline, option 1, would be better and more like what's specified in the report source.

[merks]

If you do Open Image in New Tab it shows the content:

If I copy the content into a file, including adding the XML header, it still renders like this:

Only if the file extension is .svg does it render correctly:

[speckyspooky]

@merks
I need your mind because I have 2 ways to solved it but which one is the better way (?)

Option 1) Restore the behavior like BIRT 4.14 with embedded SVG directly into the preview - 2 changes

• changes are very small

Option 2) Change the behavior to get the correct file extension and use the file integration - 1 change

• special case for the file extension creation
• only "SVG" necessary because the embedded raster-images can be read directly without file extension

Attached the screen of the 2 changes on my notepad for compare.
My favorit would be "Option 2)" but I'm completely free and need a second opinion.

[speckyspooky]

PR #1605 created to fix the issue.

[speckyspooky]

The retest with the nightly build was successful and the images will be displayed according to BIRT4.14