Created attachment 58613 [details] Testing file How to reproduce ---------------- 1. Open attached SVG file. 2. Insert attached SVG file into the same document ("Insert->Picture->From file..."). Expected results ---------------- Both pictures look the same. Results with lodraw 3.5.1 release --------------------------------- 1. stroke-dasharray was ignored. 2. Line marker was placed separately at 0,0 position.
Created attachment 58614 [details] Screenshot of the file opened and inserted in lodraw 3.5.1
Thanks for bugreport result on Fedora 64 bit: in 3.3.4: Insert->Picture->From file and File->Open shows bug (both lines solid, arrow in upper left corner of page, no text) in 3.5.4: Insert->Picture->From file works correctly File->Open show bug (Windows not tested, expected the same result)
*** Bug 64460 has been marked as a duplicate of this bug. ***
Created attachment 118550 [details] How it looks in Libreoffice 5.0.1.2 This issue is still present in Version: 5.0.1.2 Build ID: 81898c9f5c0d43f3473ba111d7b351050be20261 Locale: es-ES (es_ES) on Windows 7 (64-bit)
Migrating Whiteboard tags to Keywords: (filter:svgOpen)
Created attachment 122919 [details] The default fill color black is not used in markers. Please write a new bug report, in case dashes are rendered wrong, so that this issue is only about the missing marker. Two different problems in one report cannot be handled well. The marker is missing, because it has no fill attribute. The property 'fill' has the initial value 'black', but LibreOffice treats it as 'none'. In the middle example in the attached file, the marker should be totally filled black.
*** Bug 87440 has been marked as a duplicate of this bug. ***
Edit subject to reflect the root cause
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=644ace4143fe2576bfd71c8ab3da9d666d6523bb tdf#47446: SVGIO: Set marker's fill attribute to black... It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Created attachment 123434 [details] fill inherited I think, that the solution is wrong. It prevents, that an inherited fill can be used. In the attached example the marker in the middle inherits its fill from the ancestor, so it should be filled yellow.
Hi Regina, Thanks for reviewing. I didn't take that case into account, I'll fix it. However, it didn't work before my patch either.
Created attachment 123491 [details] fill inherited from g
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=1868a62a32cff30e3ba214cbd03917d2c6518073 tdf#47446: SVGIO: Take markers' parents into account too It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=de32fe50299a8891102528379e291919d2b66f56 SVG: Add unittest for tdf#47446 and tdf#98599 It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.