Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Furtive patach syllables with sof pasuq incorrect isClosed #105

Closed
charlesLoder opened this issue Mar 10, 2023 · 2 comments
Closed

Furtive patach syllables with sof pasuq incorrect isClosed #105

charlesLoder opened this issue Mar 10, 2023 · 2 comments
Milestone

Comments

@charlesLoder
Copy link
Owner

The problem
When a syllable with a furtive patach also has a sof pasuq \u{5C3}, the isClosed property is to true the opposite of what it normally is.

Some consideration
A syllable with a furtive patach is actually closed.

The current way of doing things is semantically incorrect but consistent.

Options

  1. Fix it so that a furtive patach syllable with a sof pasuq has isClosed set to false

  2. Update the syllabifier so that furtive patach syllables:

  • have the text actually be correct (i.e. patach + chet , not like the current chat + patach)
  • set the isClosed property correctly
@charlesLoder
Copy link
Owner Author

See this issue

@charlesLoder charlesLoder changed the title Furtive patach syllables with _sof pasuq_ incorrect isClosed Furtive patach syllables with sof pasuq incorrect isClosed Mar 10, 2023
@charlesLoder charlesLoder added this to the v0.15.0 milestone Apr 14, 2023
@charlesLoder
Copy link
Owner Author

Closing this as not planned.

The issue that sparked this was that a furtive patach followed by a sof pasuq (or potentially a colon if that is what was used), e.g. "רֽוּחַ׃" would not transliterate correctly.

  • Expected: rûaḥ
  • Received: rûḥa

The original thought was that furtive patach syllables should be closed, enabling the reverse of these lines — i.e. could check for syl.isClosed instead of !syl.isClosed.

The idea to switch how a furtive patach is encoded is a bad idea.

The furtive patach is currently encoded as "chet/ayin/he + patach"/

The idea was to encode it as "patach + chet/ayin/he".

This is a bad idea b/c no font knows how to render that encoding correctly.

@charlesLoder charlesLoder closed this as not planned Won't fix, can't repro, duplicate, stale Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant