Skip to content

getText needs to do better with newlines  #92

Open
Open
getText needs to do better with newlines #92

Description

@balmas
balmas
opened on May 2, 2016

given xml like the following:

 <div3 type="section" n="11">
                  <p>ἦν μὲν δὴ τρία μέρη τὰ κρατοῦντα τῆς πολιτείας, ἅπερ εἶπα πρότερον ἅπαντα· οὕτως δὲ πάντα
κατὰ μέρος ἴσως καὶ πρεπόντως συνετέτακτο καὶ
διῳκεῖτο διὰ τούτων ὥστε μηδένα ποτʼ ἂν εἰπεῖν
δύνασθαι βεβαίως μηδὲ τῶν ἐγχωρίων πότερʼ ἀριστοκρατικὸν τὸ πολίτευμα σύμπαν ἢ δημοκρατικὸν
ἢ μοναρχικόν.
</p>
               </div3>

it should normalize newlines into spaces so that words separated by new lines don't get mushed into one.

Activity

balmas
added a commit that references this issue on May 2, 2016

try not to smoosh lines. Workaround for Capitains/Sparrow#92

cc9eec0

balmas commented on May 17, 2017

@balmas
balmas
on May 17, 2017
ContributorAuthor

Related - it should not smoosh words from two separate elements together.

E.g. getText on

<l n="1">ἄνδρα μοι ἔννεπε, μοῦσα, πολύτροπον, ὃς μάλα πολλὰ</l><l n="2">πλάγχθη, ἐπεὶ Τροίης ἱερὸν πτολίεθρον ἔπερσεν·</l>

Produces

ἄνδρα μοι ἔννεπε, μοῦσα, πολύτροπον, ὃς μάλα πολλὰπλάγχθη, ἐπεὶ Τροίης ἱερὸν πτολίεθρον ἔπερσεν·

But should produce

ἄνδρα μοι ἔννεπε, μοῦσα, πολύτροπον, ὃς μάλα πολλὰ πλάγχθη, ἐπεὶ Τροίης ἱερὸν πτολίεθρον ἔπερσεν·
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions