sillsdev / ptx2pdf

XeTeX based macro package for typesetting USFM formatted (Paratext output) scripture files
23 stars 8 forks source link

ptxprint.sty contains unmodified styles #884

Open markpenny opened 1 year ago

markpenny commented 1 year ago

Even though I didn't have a ptxprint.sty file when I opened a project, and didn't edit any styles, PTXprint went and made a ptxprint.sty file with a bunch of styles but no settings different from the defaults.

I would expect it to ONLY be created if styles have been modified; and/or contain modified settings only. [remove .txt extension from this attahment]. ptxprint.sty.txt

mhosken commented 1 year ago

What fields were included? I suspect that the formatting of the field value differs slightly from what is in ptxprint.sty or some such. All we have to do (TM) is to make the, test equivalent.

On Wed, 2 Aug 2023, 06:53 Mark Penny, @.***> wrote:

Even though I didn't have a ptxprint.sty file when I opened a project, and didn't edit any styles, PTXprint went and made a ptxprint.sty file with a bunch of styles but no settings different from the defaults.

I would expect it to ONLY be created if styles have been modified; and/or contain modified settings only. [remove .txt extension from this attahment]. ptxprint.sty.txt https://github.com/sillsdev/ptx2pdf/files/12236511/ptxprint.sty.txt

— Reply to this email directly, view it on GitHub https://github.com/sillsdev/ptx2pdf/issues/884, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLMO3IFGOAE7FEFM2QDOLTXTHTORANCNFSM6AAAAAA3AXVTFU . You are receiving this because you are subscribed to this thread.Message ID: @.***>

markpenny commented 1 year ago

Most of them are because of a \Description field (and now I can see why -- see below). But there are also a handful of other markers: \Endmarker liv1* \Occursunder id periph \Rotation l \SpaceBefore 2 \TextProperties level_1 paragraph poetic publishable vernacular

image

This is due to us conflating \ili and \ili1 so that the desc from one ends up in the other: image

markpenny commented 11 months ago

@mhosken Any more thoughts on how to solve/fix this one?