sillsdev / sillsdev/appbuilder-pwa

Multi-word Glossary Entries not recognized

Open
#528 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
TypeScript
Stars
14
Forks
39
Avg merge
4h 50m
Merged PRs (30d)
5

Description

Multi-word glossary entries don't get identified. When a glossary entry has more than one word, it doesn't get the glossary
class=k markers
Screenshot 2024-05-11 at 2 30 56 PM
which means, multi-word glossary entries aren't working in text. When in text, there is a multi-word phrase with the glossary span, it points to a no-existent entry in the glossary so displays nothing. Also, in some of these cases, the glossary entry gets repeated in the text itself
Screenshot 2024-05-10 at 10 26 16 AM
Screenshot 2024-05-10 at 10 26 26 AM
https://media.ipsapps.org/krr/osa/bible/

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the issue in the linked PWA with a glossary entry containing multiple words, using the screenshots as a reference for the missing class marker, nonexistent glossary entry, and duplicated text. Trace the glossary recognition and in-text display flow; done means multi-word entries are recognized, link to the correct glossary entry, and are not duplicated.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.