marcus/scour
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix whitespace handling for SVG 1.2 flowed text

Browse source 
See 718748ff22

Fixes https://github.com/scour-project/scour/issues/235
This commit is contained in:
Patrick Storz 2020-05-17 17:33:50 +02:00
parent f65ca60809
commit 40753af88a
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
scour/scour.py
View file

@ -3354,6 +3354,10 @@ def chooseQuoteCharacter(str):
return (quote, hasEmbeddedQuote)
TEXT_CONTENT_ELEMENTS = ['text', 'tspan', 'tref', 'textPath', 'altGlyph',
'flowDiv', 'flowPara', 'flowSpan', 'flowTref', 'flowLine']
# hand-rolled serialization function that has the following benefits:
# - pretty printing
# - somewhat judicious use of whitespace
@ -3437,7 +3441,7 @@ def serializeXML(element, options, indent_depth=0, preserveWhitespace=False):
# "text1\ntext2" and
# "text1\n text2"
# see https://www.w3.org/TR/SVG/text.html#WhiteSpace
if preserveWhitespace or element.nodeName in ['text', 'tspan', 'tref', 'textPath', 'altGlyph']:
if preserveWhitespace or element.nodeName in TEXT_CONTENT_ELEMENTS:
outParts.append(serializeXML(child, options, 0, preserveWhitespace))
else:
outParts.extend([newline, serializeXML(child, options, indent_depth + 1, preserveWhitespace)])
@ -3448,7 +3452,7 @@ def serializeXML(element, options, indent_depth=0, preserveWhitespace=False):
if not preserveWhitespace:
# strip / consolidate whitespace according to spec, see
# https://www.w3.org/TR/SVG/text.html#WhiteSpace
if element.nodeName in ['text', 'tspan', 'tref', 'textPath', 'altGlyph']:
if element.nodeName in TEXT_CONTENT_ELEMENTS:
text_content = text_content.replace('\n', '')
text_content = text_content.replace('\t', ' ')
if child == element.firstChild: