| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
When width or height is specified in percent, it is supposed to be
relative to the viewport.
https://www.w3.org/TR/SVG/coords.html#Units
Task-number: QTBUG-2279
Change-Id: Ia1597b88d4e1c705a6755db788a18cf790d5ee97
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|
|
|
|
|
|
|
|
|
|
|
| |
In QSvgAnimateTransform::resolveMatrix(), the indexes into the vals
array were previously calculated to be 1,1 in the final step, where
currentPosition=1, by using qCeil()/qFloor(), resulting in rotationDiff=0
and the rotation snapping back.
Fix by setting startElem to endElem - 1 when available.
Task-number: QTBUG-67525
Change-Id: Ie1d7e8be6ea8dd9aa155ee8c479037bc269f6df7
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|
|
|
|
|
|
|
| |
QString allocation can be avoided in some places after
the introduction of QStringRef::toInt()/toDouble().
Change-Id: Ib400f49df04847f85aeea0190215f864f71223bd
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|
|
|
|
|
|
|
|
|
| |
Extend existing enum by "none" and extract a function
to return the specification so that "none" can be cleanly
handled.
Task-number: QTBUG-66646
Change-Id: If4ec1056c6f0fa407f76b747634bb898e75a5474
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|\
| |
| |
| | |
Change-Id: Ic852de844e8a1e7acfe4aaaf79acf46674ba5ffa
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The linker complains:
LINK : warning LNK4281: undesirable base address 0x67000000 for x64 image; set base address above 4GB for best ASLR optimization
And it's not really required anymore, as the recommended /DYNAMICBASE is
the default.
Change-Id: I56b444f9d6274221a3b7fffd150caab1beecfd43
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
|
|\ \
| |/
| |
| |
| |
| |
| | |
Conflicts:
.qmake.conf
Change-Id: I2d1692e18205aa86e7181d907eac21dc096744ae
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In 4a9470e55ce08752501868c60762268fdf2c44af there was introduced
specific isNull() implementation for QSvgIconEngine. But the existing
check was not sufficient for the scenarios where the QIcon is
constructed by deserialization from QDataStream (from previously
serialized valid svg QIcon).
We also need to check the svgBuffers as the previously serialized
svgFiles are silently ignored by QSvgIconEngine::read().
Change-Id: Id64ce233baead053c8b5ca61f1d416e46a8c6088
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>
Reviewed-by: David Edmundson <davidedmundson@kde.org>
|
| |
| |
| |
| |
| |
| |
| | |
Use begin()/end() instead.
Change-Id: Ia51069dba42aa26e78b071b147a69c7f36947d12
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
|
|\ \
| |/
| |
| | |
Change-Id: I7e158268343c5d92d666d266a87095f1154714dd
|
| |\
| | |
| | |
| | | |
Change-Id: I2c5f1f592e2141b93fa274124a96f2a63b83acdb
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Referring to an ancestor in a <use> element would lead to endless
recursion. Add checks to avoid recursion, and also emit a warning
while parsing.
Task-number: QTBUG-64425
Change-Id: I9ee1b9bfef13796cc3f387ff8579c6b13bc4ae9a
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
|
| |\ \
| | |/
| | |
| | |
| | |
| | |
| | | |
Conflicts:
.qmake.conf
Change-Id: I8b9a5a395dbd6659f7806eeebdd665c760a71ec1
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: I2a2501a61107717b8e1adacad064664cf54cd972
Reviewed-by: Jani Heikkinen <jani.heikkinen@qt.io>
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: I3917e7ab180b49c77bff0d071eca10f8bc7f33c7
Reviewed-by: Jani Heikkinen <jani.heikkinen@qt.io>
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: Iea5521d1fd53d5af21f99023f8d9ed15ef2f4f68
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Change-Id: I82fe38e2fbcc070d02d2acb34a43a8c971e84ae9
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: I2a3c0a3734548919d8a337b88b9aa6fdf725bbc1
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
QSvgGenerator::metric returned incorrect value for
`PdmDevicePixelRatioScaled` resulting in degenerate
`devicePixelRatioF()` return value.
[ChangeLog][][QSvgGenerator] Report correct devicePixelRatioF
Task-number: QTBUG-63159
Change-Id: Ib45e930b6b1b6d85aea5a68dd2b1b5f16a4eed3b
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: I9a18a0cfc86411336cbf29e0b7fb2f77f704d284
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|/ /
| |
| |
| |
| | |
Change-Id: Ibaae1afa39414a951a57c36037fa3b951654329f
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
|
|\ \
| |/
| |
| |
| |
| |
| | |
Conflicts:
.qmake.conf
Change-Id: I1ebd40264cf1afaf5bcbf8b2f272e153f51a5625
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Sequencing of a matrix multiplication seems to be erroneous, this may
lead to incorrect svg rendering under certain circumstances
Task-number: QTBUG-61568
Change-Id: Id8b6c425d4affde8c71faad455a63a87ef5e13a4
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If the QImage given to read() already has the right size and depth, we
don't need to allocate a new image. This is consistent with other
image decoders and why a QImage is provided to the method in the first
place.
Change-Id: I4944211d4a8d4f19d000cace4a48df505d375f52
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
| |
| |
| |
| |
| |
| |
| | |
Task-number: QTBUG-60583
Change-Id: I685d4d68663703e220150869dff671718a8190a1
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|\ \
| |/
| |
| |
| |
| |
| | |
Conflicts:
.qmake.conf
Change-Id: Idf7f59a8c4b78438b496f2c16ba9ce48573042e9
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When a stroke-dasharray had multiple values separated by spaces it would
end up combining them into one instead of treating them separately. This
ensures that it is correctly respected.
[ChangeLog] Parses the stroke-dasharray attributes in a SVG file
correctly.
Change-Id: If6e9babe9f63637e56d3d1dba914557829f9d5f0
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|\ \
| |/
| |
| | |
Change-Id: I855627bf0387be820d97bbc0226fed07fd1db64a
|
| |\
| | |
| | |
| | | |
Change-Id: I4b41cae71f5a5f734a7e3bb8b95130b0cb35e0ac
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Use method which returns qreal. It is available since Qt 5.6.
Task-number: QTBUG-59729
Change-Id: Ifc1a101c8dd837597c35ce14f0314218d2882501
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: Ifdb27c6fed26f95acff30f64259d6905e88c3a6f
Reviewed-by: Paul Olav Tvete <paul.tvete@qt.io>
|
| |\ \
| | |/
| | |
| | | |
Change-Id: I9244a68bf4709fa136c52132a66d70d7c0b8897b
|
| | |\
| | | |
| | | |
| | | | |
Change-Id: If6f9147b14fcf698e707069a7334404dfc21be14
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Change-Id: I7495086a3712ef817380c80d3dea7d49da047ed8
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The code would generate radial gradiants named with an "xml:id"
attribute, in contrast to linear gradients which is named by an "id"
attribute. Those two forms should be equivalent according to the spec,
but in practice most browsers and viewers only accept the latter.
Change-Id: I798387bdbd3a6ea831af8ce103683141b567e940
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Pattern brushes was not implemented in the svg generator. Shapes
drawn with such brushes would not be included in the svg file.
[ChangeLog][][QSvgGenerator] Add support for pattern brushes
Task-number: QTBUG-58148
Change-Id: Ib275661c596631fea64cb250c9743a529cd7b834
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The raster engine accepts and renders non-normalized rectangles. Make
the svg paint engine do the same, instead of just ignoring them.
Task-number: QTBUG-58145
Change-Id: Ifa82fd580574bc908552efbb1eeace21bde76ed7
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add a function printing file name and location.
Could not resolve property : linearGradient4538
becomes
/usr/share/icons/breeze/actions/24/document-new.svg:123: Could not resolve property: linearGradient4538
Task-number: QTBUG-52079
Change-Id: I843f65eb4ca517d103031f9921a16090a6ff6720
Reviewed-by: David Faure <david.faure@kdab.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
It is then possible to suppress warnings by
setting for example QT_LOGGING_RULES to qt.svg.warning=false.
This will suppress the warnings floods which are currently
produced by faulty theme svg icons on KDE 5 systems.
Task-number: QTBUG-52079
Change-Id: I50e3c2e3eab610e182293c8938c6188694eabfb7
Reviewed-by: David Faure <david.faure@kdab.com>
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This was via a virtual hook in QIconEngine in Qt 5.7
This is especially useful for the SVG icon loader which returns no
availableSizes which means we currently have no way to tell if an icon
was loaded or not.
Change-Id: Ic14971f94213fb2309f29bd0daa0a1c387d4da3c
Reviewed-by: Kai Uwe Broulik <kde@privat.broulik.de>
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | |
| | | |
Conflicts:
.qmake.conf
Change-Id: I4965154ceaebc75fa968417f3cba47582574a01d
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
If there is no stylesheet set then this is a no-op, it ensures that this
much of the stylesheet is still utilized when one is set.
Change-Id: I6f3c15b2ddf89d8e4471acf094a48141ff06d511
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The viewBox attribute defines the svg internal coordinate system. If no
width/height is set, the viewBox size is the right choice for defaultSize.
Otherwise the standard prescribes that the specified height/width must be
used.
The old behavior ignored the width and height attributes, and this caused a
lot of problems in qml. If the viewBox was very small the result was a low
resolution image. If the viewBox was very large, loading took ages and the
result used much more memory than needed. Both situations could be avoided
by setting sourceSize. But when using the same image several times, the
sourceSize must be set every time to the same value, otherwise the image
cache would not work. It is cheaper to have the same high-quality source
image in the cache, and scale it down when required.
With the new behavior it is possible to control the default image size
directly in the svg file at one place while it is still possible to set
different sourceSizes if needed.
Task-number: QTBUG-44863
Change-Id: I9c2fc7c122a29ebcf288b7cbd12427e081d404d5
Reviewed-by: André Klitzing <aklitzing@gmail.com>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
|
|/
|
|
|
|
| |
Change-Id: I9c6bc47803b5decb47787ad69f42259e85470f57
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
|\
| |
| |
| | |
Change-Id: I43247261a4dc8badc4ad4f4d2137cdbd499f29db
|
| |\
| | |
| | |
| | | |
Change-Id: Ibcca419da389fc7538ee050de7c59e8b400e1978
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Change-Id: Ib15aa96c2ed84cf3ce9793385476f9fb3cacec8c
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Change-Id: I3cf5d9b6d6212128adab4682de33b192672f7c1f
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The imageformats plugin was trying to pass them off to QSvgRenderer,
which can't handle them when QT_NO_COMPRESS is defined.
Change-Id: I92241370d8fd4ed2328fb13a01306fa8a7e5ef6e
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Eirik Aavitsland <eirik.aavitsland@qt.io>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Remove QIconEnginePlugin::keys() and QImageIOPlugin::keys():
they are not overriders since Qt 5.0.
Change-Id: I9127b5b72c9c37b450c6f0d0095e47b8d5454322
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>
|