summaryrefslogtreecommitdiff
path: root/svgio
AgeCommit message (Collapse)Author
2016-04-12SVG: Check that masking-path-07-b.svg doesn't hangXisco Fauli
Change-Id: If8b9a0dd955611f8ebb2859fa9b10a61bfb2230b Reviewed-on: https://gerrit.libreoffice.org/24010 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-12SVG: actually this code is unnecessary after...Xisco Fauli
... f5d4a688346a939a58b1df69d771dede177b3370 Change-Id: I2911763c38a6be275bcd7ae0c9a51cd7a2e9fe4a Reviewed-on: https://gerrit.libreoffice.org/24012 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-12SVG: Add unittest for #i125329#Xisco Fauli
Change-Id: I2d6fbd81c8d95283cf8937739e338ddac14d8df0 Reviewed-on: https://gerrit.libreoffice.org/24008 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-12SVG: Clean up this code a litle bitXisco Fauli
Change-Id: Ib941321d43a9a04d57f3c7c4a55528927fed5036 Reviewed-on: https://gerrit.libreoffice.org/24009 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-04tdf#97499 Fixed containers parameters clearing #4tymyjan
Change-Id: I7c96181399f4d7e62d4aceca404b22d68f903513 Reviewed-on: https://gerrit.libreoffice.org/23754 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-04tdf#97499 Fixed containers parameters clearing #3tymyjan
Change-Id: I1c105de7e1df251533179fba77e3797ef1c4e2d4 Reviewed-on: https://gerrit.libreoffice.org/23752 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-03SVG: Using the marker property from a style sheet is equivalent...Xisco Fauli
... to using all three (start, mid, end). However, shorthand properties cannot be used as presentation attributes. Change-Id: Ie3842b8fa87f562f038600f8f012d8434363b3cd Reviewed-on: https://gerrit.libreoffice.org/23749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-04-01tdf#97966 Drop 'static' keywordsWastack
Including no keywords from extern "C" blocks Change-Id: I364aa7d67c2bf867588798e07df95c3f3f2bd7f3 Reviewed-on: https://gerrit.libreoffice.org/23676 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-30tdf#97657: SVGIO: Don't iterate over parent's clip-paths...Xisco Fauli
... when it's a clip-path content Change-Id: I222b429a61409d9670b9d60357895542a15afc15 Reviewed-on: https://gerrit.libreoffice.org/23624 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-29SVGIO: Don't inherit style attributes from parent if the...Xisco Fauli
... shapes define their own attributes. Partially revert commit c2d130425cc876cceb96cc7e47636426a9e48fa8 Change-Id: Iee98003f4ff63575a28818b091ab77431337192f Reviewed-on: https://gerrit.libreoffice.org/23571 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-26SVGIO: Shapes within a clippath element might inherit...Xisco Fauli
... fill, stroke, stroke-width and others attributes from parent element, like use element Change-Id: I911bc50409428e81135b81e6dfc82acd1c21d113 Reviewed-on: https://gerrit.libreoffice.org/23527 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-17tdf#98599 SVG: consider attributes of 'defs' elementRegina Henschel
The element 'defs' is not rendered, but children of it may inherit the attributes. Therefore these has to be considered. Change-Id: Ic6c21645439731db63f22adf5a5a4e2af2b74b71 Reviewed-on: https://gerrit.libreoffice.org/23309 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-03-11tdf#47446: SVGIO: Take markers' parents into account tooXisco Fauli
Change-Id: I41b7228ab5980581ab5e1c8c4d4d466da6045086 Reviewed-on: https://gerrit.libreoffice.org/23124 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-10tdf#97539: SVGIO: iterate over parent's clippathsXisco Fauli
Change-Id: Iadb335bf5b328fa3fd9683da5433c299e62afa8e Reviewed-on: https://gerrit.libreoffice.org/23106 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-10Extract Directories from BootstrapFixtureBaseStephan Bergmann
(as some tests derive from the latter only for the Directories part, not for the setUp/tearDown overrides: those tests will be cleaned up next) Change-Id: Ib6b78eea868b8bc21d4cc6e8fd9e1d025deca05f Reviewed-on: https://gerrit.libreoffice.org/23078 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-03-09tdf#47446: SVGIO: Set marker's fill attribute to black...Xisco Fauli
... if it's not set Change-Id: Iefec59f885c3708defec2b636836617bf97387fc Reviewed-on: https://gerrit.libreoffice.org/23046 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-09SVGIO: Fix problem when style's parent contains a mask elementXisco Fauli
Change-Id: I26aab6c0f959f6cc78d040f59f1acf0c7e29931b Reviewed-on: https://gerrit.libreoffice.org/23045 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-08Updated to solve problem with gerrit patch 22363Jan Iversen
Regina posted comment on 22363, that this change was needed. Change-Id: I9021d40ee7c1ca7ca74d67c61fe58ce976da38c0
2016-03-07Revert "tdf#97538: SVGIO: Add unittest"Xisco Faulí
It fails in Tinderbox Linux-rpm_deb-x86@71-TDF This reverts commit 3c465c259c617e30b01c5ea25b5203a605cd4269. Change-Id: I20fc69dd785354cc6567b73251ee9161a811bdf0 Reviewed-on: https://gerrit.libreoffice.org/22987 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-07SVGIO: tdf#97539: clip-path elements might contain...Xisco Fauli
reference to other clip-paths Change-Id: I3722b31cefa4df6225e369b3d1db9f46be5933ff Reviewed-on: https://gerrit.libreoffice.org/22956 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-07tdf#97538: SVGIO: Add unittestXisco Fauli
Change-Id: I5bf523ac532ad19a005d99e19deb1e0ec9ce96e4 Reviewed-on: https://gerrit.libreoffice.org/22955 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-06SVGIO: Check minX, minY, maxX and maxY in polypolygonXisco Fauli
Change-Id: I768bef34ad0ba7708134e73981dabf0d47c2f259 Reviewed-on: https://gerrit.libreoffice.org/22952 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-03SVGIO: Check height and width of polypolygoncolorXisco Fauli
Change-Id: Ibc5844d4e4a89fae7bbc8d2b854384d0c433500f Reviewed-on: https://gerrit.libreoffice.org/22883 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-03tdf#98113: SVGIO: Add SvgANode class to handle transport ...Xisco Fauli
... attribute in anchor elements. I used Svggnode class a a reference for this. Change-Id: Id2a58bd913f9984dc91163ca0f333c016aa981f1 Reviewed-on: https://gerrit.libreoffice.org/22822 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-03-01loplugin:unuseddefaultparam in svgioNoel Grandin
Change-Id: I47fe88cd8c943acb9162cdfca53fb5fdbaacec04
2016-03-01fix typoXisco Fauli
Change-Id: I3e1e28269d8b2d6d45bba6bc43aa8f20c1963956 Reviewed-on: https://gerrit.libreoffice.org/22793 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2016-02-28tdf#98119 Allow markers on zero-length linesRegina Henschel
Inkscape, Seamonkey, Chrome and Batik show markers on zero- length lines, LibreOffice should do that too. The patch removes the restriction start<>end. Change-Id: I984baf755df7b2689e00d2a23b40e3f059c0add0 Reviewed-on: https://gerrit.libreoffice.org/22740 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2016-02-27Revert "SVGIO: tdf#97539: clip-path elements might contain..."Xisco Faulí
This reverts commit 7b7ffedfa6357b902a5e354d986dac39a3b2f462. Change-Id: I018830e17cb83c5fe3574b76939673877ac2004d Reviewed-on: https://gerrit.libreoffice.org/22726 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-25tdf#97936: SVGIO: Add unittestXisco Fauli
Change-Id: I7adb55dc408811f03365578f815bf1b7fbe7f2fd Reviewed-on: https://gerrit.libreoffice.org/22679 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-25Partially revert b6a62f07557fe4ae0931e0f3e0f306d2f69b116bXisco Fauli
it has been fixed by 423b79e7366203db3f57dea75b8cb9eb852b5614 in a better way Change-Id: Ia3192071f00522ac111d3e95e4dc05dc57a6267e Reviewed-on: https://gerrit.libreoffice.org/22677 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-21tdf#97538 no cropping if width, height and viewBox missingRegina Henschel
Now the whole image is inserted, so that it can be cropped in the application. The patch repairs in addition a wrong ratio in the case height is missing but viewBox is given, and fixes some comments. Change in SvgImportTest.cxx by Xisco Fauli. Change-Id: I8c0fbf8b78cf7ab5bc97d04783b49892da3d2849 Reviewed-on: https://gerrit.libreoffice.org/22363 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-19new loplugin: find write-only fieldsNoel Grandin
Change-Id: I0f83939babacf92485420ee63f290a297d7cb717 Reviewed-on: https://gerrit.libreoffice.org/22498 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-19SVGIO: Fix problem when relative font-size depends on its...Xisco Fauli
... parent's font-size Change-Id: I13943923c401a5a06b3aa51181fd75292534e888 Reviewed-on: https://gerrit.libreoffice.org/22452 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-18tdf#97752 SVGIO ignore not matching parent paintsJochen Nitschke
fill attributes can have ONE of 3 kinds of paints: * pain colour fills * gradient fills * pattern fills if element has a fill we should not get fills of an other kind from parent Change-Id: I376a8fea4a7185b037a069a28191b7f37f6788bd Signed-off-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-on: https://gerrit.libreoffice.org/22273 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-18SVGIO: tdf#97941: Don't double tspan fontsizeXisco Fauli
Same as in 701324a1e1f7e0c181ff1a50956ced686785ea53 Change-Id: I531bef4821008ef71951506c133f999b9ab4f4ff Reviewed-on: https://gerrit.libreoffice.org/22450 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-17SVGIO: tdf#97539: clip-path elements might contain...Xisco Fauli
reference to other clip-paths Change-Id: I0c426c892da2478293b78ea93db087d781275896 Reviewed-on: https://gerrit.libreoffice.org/22411 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-13SVGIO: tdf#97659: Add support for RGBAXisco Fauli
Change-Id: Icbf3796cdc95f91d298a5ca52d44931b3985eac6 Reviewed-on: https://gerrit.libreoffice.org/22303 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-11SVGIO: tdf#97543: Add import testXisco Fauli
Change-Id: Ia89f43ed1da9ead254f191bcbcc4ac338a9dda7a Reviewed-on: https://gerrit.libreoffice.org/22302 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-11tdf#97542: SVGIO: First check the current fill attributeXisco Fauli
Differenciate when gradient is defined in the text element or in its parents Add unit tests Change-Id: I85b09f72fbb1a3c8d2fa7350b4396b5bf924277f Reviewed-on: https://gerrit.libreoffice.org/22131 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-09SVGIO: tdf#79163: Add import testXisco Fauli
Change-Id: I891425f331e9be21c160eb0b11f05dba280bef23 Reviewed-on: https://gerrit.libreoffice.org/22196 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-09SVGIO: tdf#85770: Add import testXisco Fauli
Change-Id: Ifffe1d461ae6fa30a9cc61df2ece5716078031ab Reviewed-on: https://gerrit.libreoffice.org/22190 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-09SVGIO: tdf#45771: Add import testXisco Fauli
Change-Id: Ib1b66a3d91374bcf458bb368483f33cd4b8e0f0d Reviewed-on: https://gerrit.libreoffice.org/22189 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-09SVGIO: Add import test for 7c151f39144b7401a99c7b736a41e5dce4ac4db1Xisco Fauli
Change-Id: I4ab4463dfa2422fd0d8fa1fd65d7765f716c9bc3 Reviewed-on: https://gerrit.libreoffice.org/22188 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-02-09Remove excess newlinesChris Sherlock
A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-09SVGIO: Add import test for Fontsize keywordsXisco Fauli
Change-Id: I042a1deeec1c07f73f09ddd2fb71b9d2eba6b8c6 Reviewed-on: https://gerrit.libreoffice.org/22187 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tomaž Vajngerl <quikee@gmail.com> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-02-08SVGIO: Return default font size if size value is negativeXisco Fauli
Change-Id: Iec18c6511b57c4e61047055b53589f71dda3d601 Reviewed-on: https://gerrit.libreoffice.org/21972 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>
2016-02-08loplugin:unusedmethodsNoel Grandin
using an idea from dtardon: <dtardon> noelgrandin, hi. could you try to run the unusedmethods clang plugin with "make build-nocheck"? that would catch functions that are only used in tests. e.g., i just removed the whole o3tl::range class, which has not been used in many years, but htere was a test for it... <noelgrandin> dtardon, interesting idea! Sure, I can do that. Change-Id: I5653953a426a2186a1e43017212d87ffce520387 Reviewed-on: https://gerrit.libreoffice.org/22041 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-07tdf#97543 SVGIO: Rework getVisibility()Xisco Fauli
Change-Id: Idb3eaa27e3e8ec177ebd2bad81f53c4d3da9d241 Reviewed-on: https://gerrit.libreoffice.org/22169 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-05SVGIO: tdf#97383: Keep ratio in embedded images...Armin Le Grand
... when height and width are different SVG specifies embedded graphics to keep their aspect ratio (thanks to Regina for pointing out). To implement this for Office SVG import, I had to correctly set the graphic data unit dimensions to make the existing SvgAspectRatio mapping work. All together this now simplifies this method and packs all in clean linear transformations in the created stack of drwainglayer primitives. Change-Id: Id80c2f74ffb148085085b2c9627bc31bc15bdee5 Reviewed-on: https://gerrit.libreoffice.org/21830 Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de> Tested-by: Armin Le Grand <Armin.Le.Grand@cib.de>
2016-02-01tdf#79163: SVGIO: Fix problem with opacity attributeXisco Fauli
Opacity attribute didn't work because it was always set to 1.0 Change-Id: I3a2029ef23acf9d8c0f346e04341db33c6802b8e Reviewed-on: https://gerrit.libreoffice.org/21911 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>
generated by cgit (git 2.34.1) at 2024-08-11 14:42:27 +0000