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

[Crum/References] Revisit Nag Hammadi References #167

Open
pishoyg opened this issue Aug 11, 2024 · 1 comment
Open

[Crum/References] Revisit Nag Hammadi References #167

pishoyg opened this issue Aug 11, 2024 · 1 comment
Labels
maintenance What: Small fix UI Why: Better user interface user Why: User convenience

Comments

@pishoyg
Copy link
Owner

pishoyg commented Aug 11, 2024

  1. It seems that they always have the word ext. Eliminate it.
  2. Consider including them in the Flashcards HTML (currently you're using word-parsed-no-ref). Perhaps as a footnote.
@pishoyg pishoyg changed the title [Crum/Flashcards/HTML] Reconsider the Use of word-parsed-no-ref for the Full Entry [Crum/References] Revisit Nag Hammadi References Aug 11, 2024
@pishoyg pishoyg added the user Why: User convenience label Aug 11, 2024
pishoyg added a commit that referenced this issue Aug 11, 2024
pishoyg added a commit that referenced this issue Aug 11, 2024
Classes are desirable for the JavaScript behaviour.
We also want the references.
pishoyg added a commit that referenced this issue Aug 11, 2024
Pick up the new "word-parsed-classify" columna, and remove
"word-parsed-no-ref".
pishoyg added a commit that referenced this issue Aug 11, 2024
Until we have time to write the JavaScript to convert them to footnotes.
@pishoyg
Copy link
Owner Author

pishoyg commented Aug 11, 2024

They don't look that bad with 50% opacity actually! We could keep them as is, not footnotes!

@pishoyg pishoyg added the UI Why: Better user interface label Aug 17, 2024
@pishoyg pishoyg added the maintenance What: Small fix label Sep 9, 2024
@pishoyg pishoyg added this to coptic Sep 11, 2024
pishoyg added a commit that referenced this issue Oct 15, 2024
They're too obtrusive. We will hide them for now, and let's think about
them later.

We also make them visible in `dev` mode, so we can see them when we want
to.
pishoyg added a commit that referenced this issue Dec 10, 2024
This reduces jitter. Otherwise, if they are visible by default and only
get hidden by JavaScript, they will jitter on page load.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance What: Small fix UI Why: Better user interface user Why: User convenience
Projects
Status: No status
Development

No branches or pull requests

1 participant