-
-
Notifications
You must be signed in to change notification settings - Fork 605
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
Fix footnotes in Flee Mortals #1869
Conversation
Use the footnoteEntries attribute in spellcasting to explain symbols for casting time
"Fix" is a strong word; what, in particular, is wrong with the current version? The |
I concede; "Fix" is a strong word. "Propose alternate way to do this" is better. Most of the time, I can make beautiful markdown notes out of 5eTools JSON data. This particular structure causes problems.
I could make assumptions that any footnotes with the same key will have the same value and shove them into a map and render them before or after So I thought I would suggest this approach instead. You still have the casting time indicators in the spell list, using superscript. And there is a single element in the As of now, this is the only file in the Homebrew repo that uses this structure. So I figured now would be the time to propose an alternative. ;) |
For context, ebullient makes the 5eTools (& PF2e) to Obsidian Markdown tool. |
ty I did remember this time
"Hate" is another strong word, but I do like my nice hover-able superscript letters. That said, on closer inspection, the I think my path of least resistance here would be to make it into a loadable entity type i.e. "footnoteEntry": [
{
"name": "Spell Action Note",
"source": "FleeMortals",
"page": 123,
"entries": [ ... ]
},
...
] and then "{@spell detect magic}{@footnote {@sup A}|Spell Action Note|FleeMortals}, {@spell disguise self}{@footnote {@sup A}|Spell Action Note|FleeMortals}" This would fix the Plutonium jank (For Free:tm: by way of the existing hover-able tag handling paths), and de-dupe all the footnote strings. I imagine you could then decide how you wanted to deal with linked Thoughts? |
=) Fair enough. The hover is definitely nice for the web UI, it just doesn't work well in Obsidian or straight markdown.
Does this break existing footnotes (I think you're saying it does, and you want to change all of the existing footnotes?):
For example:
Can we do something like (that would mean you could leave all other uses of |
yes and yes, they're trivial enough to migrate
apologies; smashed out meaningless junk before going to bed. What I meant was:
that's a definite hate. I'm not maintaining two systems for the same thing, especially for something as uninteresting as footnotes
is it really, though? Make one/or an IDE template of your choice with a generic name (say
those are likely broken too, but iirc the implementations are separate, so we can table that really what I'm asking is, "is this any different to a |
I agree, it's fine in terms of the actual technical challenge of making an entry. The pain is that in a brew with 80 unique footnotes I now need to come up with 80 source-unique I thought the point of footerEntries on a Spellcasting entry was that this is how wizards and inspired brew sources laid out the notes for spell lists? Changing the whole footnote system which is used very differently in other parts of the site to enable this specific brew feels strange. Though I understand the wish to factor out non-standard behaviour. |
I expect a "footnote" to be something like this:
so the "uniquely named entity version" of it would just be this: "footnoteEntry": [
{
"name": "Footnote 17",
"source": "MySource",
"page": 250,
"entries": [ ... ]
},
{
"name": "Footnote 18",
"source": "MySource",
"page": 250,
"entries": [ ... ]
},
...
] or for a book which doesn't uniquely number their footnotes: "footnoteEntry": [
{
"name": "Footnote 1 p250",
"source": "MySource",
"page": 250,
"entries": [ ... ]
},
{
"name": "Footnote 2 p250",
"source": "MySource",
"page": 250,
"entries": [ ... ]
},
...
]
agreed..? Not sure I follow, this is not about changing
I'll contest this:
|
as an aside I think |
Frankly, I think this is worse, and it really messes up what are otherwise functional footnotes everywhere else. This is the only source that does things this way, and I feel like you grabbed the footnote as the closest thing to what you wanted (not trying to put you on the defensive here, it did give you the nice hover behavior, so I can understand the choice).
The This particular usage (in FleeMortals) is inconsistent with the others. Technically, it's a cross-reference to a notation convention in the source material. You could do something like
I don't think this is the same thing. Maybe
It seemed footerEntries was the right place for this, which is why I suggested adding it there.
It really does. Most of those other footnotes are very contextual (none of them match each other). I agree that changing all footnotes to deal with this case (where this case is the outlier) seems strange. |
fair enough, and yes, that was what I was thinking of (e.g. wiki-like
I'll admit I'm not the one converting the brews; my cherry-picked example above just happens to be one of the PDFs I have sitting on my drive. In my head this sounded like a good idea, clearly, but apparently not @revilowaldow got any handy you could screenshot/post snippets from? E.g. I see "Exquisite Exandria" has a lot of footnotes; what's the source text look like for those? |
From this one: https://raw.githubusercontent.com/TheGiddyLimit/homebrew/master/adventure/JVC%20Parry%3B%20Call%20from%20the%20Deep.json We have text like this:
Which would appear as above. The footnote number would be generated (relative to the note), with footnotes at the bottom in the same way as shown above. The same is also true for footnotes in |
ty both, good stuff So for the recipes, As for the markdown, all looks reasonable to me. I imagine explicitly differentiating I think the question then is, as I've been shouted back from the cliff-edge on the |
Yes!
😝
I think the |
Sounds good to me, apologies if I've been shouting, 2°C Monday morning innit. I won't be satisfied until get a |
coolcool, will see to this after the new book's in and done, so will leave this PR until ~the weekend and close it when I've made the
not at all; please do always shout. I cope by CSSing a funny moustache over your avatar.
😩 |
Use the footnoteEntries attribute in spellcasting to explain symbols for casting time