viewBox not used for inline SVG ? about decktape HOT 12 CLOSED

Not that I also use svg {height: 100%;} in my css.

from decktape.

In fact viewBox is used but it seems that width and height attributes of the SVG file are used in priority over the viewBox because when I remove the SVG attributes width and height then the SVG is correctly rendered as in browser.

Any idea to make it a permanent fix ? Should I open an issue for phantomjs ?

from decktape.

A suggested in #4, that may be better to directly report this point in Qt issue tracker directly:

https://bugreports.qt.io/browse/QTBUG

from decktape.

Removing width and height attributes does not work well. Qt Webkit needs to be fixed. I filled an issue there : https://bugreports.qt.io/browse/QTBUG-46986

from decktape.

@astefanutti I'd like to test it with Qt 5.5. Is the best way replace phantomjs/src/qt source code with Qt 5.5 ? Nothing more to do ?

from decktape.

Sounds like a good idea. You can replace the phantomjs/src/qt. An other approach would be to apply the diff between 5.4.1 and 5.5.0.

There may be some PhantomJS build resources to update like:
ariya/phantomjs@68ca7c3#diff-d6d9dd3e06474412f0bb613c0b88248bR2

Let me know how it goes.

from decktape.

Well since Qt example browser give the same result as with phantomjs/decktape I won't bother switch phantomjs to 5.5. Instead I will build qt 5.5 from source with browser example and do the test from the browser example.

from decktape.

Great idea, that'd be much easier.

from decktape.

I confirm the bug is still present in 5.5.1, tested with examples/webkitwidgets/browser.

from decktape.

OK, thanks for head up.

from decktape.

The issue does not appear using QtWebEngine (tested with examples/webengine/quicknanobrowser/quicknanobrowser). And after reading some posts about QtWebKit/QtWebEngine, I saw that QtWebKit is not supported anymore... So I guess the best is to wait PhantomJS to move to QtWebEngine as they said there : ariya/phantomjs#12931

from decktape.

I close since it depends on QtWebkit.

from decktape.