srophe / caesarea-data

Data repository for Caesarea-Maritima.org
0 stars 2 forks source link

Add function in post-processing will convert " | " to a tei:lb #100

Closed wlpotter closed 1 year ago

wlpotter commented 3 years ago

Note that we are currently using the pipe character, "|", to signify line breaks in testimonia sources. We should decide how in the TEI we want to encode this and figure out a way to make this change (potentially a clever regex could handle it)

wlpotter commented 2 years ago

56 instances of "|" in our data; what do we want to replace with? <lb/> seems a natural choice?

This should be an easy find-and-replace. Do we need an app issue for display?

wlpotter commented 2 years ago
davidamichelson commented 2 years ago

Discuss with Joe future encoding and also how to display the linebreak if at all.

wlpotter commented 2 years ago

@davidamichelson quick question on this; do we want to include surrounding whitespace? Currently these are encoded " | " so we could either replace as:

Do we want a leading and trailing single space?

wlpotter commented 2 years ago

No whitespace on either side; make an issue for rendering as the start of a new line

wlpotter commented 2 years ago

App issue created: https://github.com/srophe/caesarea/issues/291

wlpotter commented 2 years ago

Note: I have moved the documentation update to a separate issue (#114) since there are several other needed revisions to that section of the documentation

davidamichelson commented 2 years ago

We can brief Joe on this when we meet on Sept 23

wlpotter commented 1 year ago

Note; we'll add this functionality to the post-processor to convert to a tei:lb

wlpotter commented 1 year ago

Fixed this in the post-processing script so that " | " is treated as a <lb/> element