-
-
Notifications
You must be signed in to change notification settings - Fork 428
-
|
The PR #380 has me notice that we still have some naming inconsistencies. I believe that we settled the debate about containing "Choose: " and that we should remove it because the real SRD name doesn't contain that. However if we keep this, I wonder why we still keep "Expertise 1" for bards, for instance, instead of "Expertise". To me, we should be as close as possible to the SRD, and nowhere does the SRD mention the "Expertise 1" or "Metamagic, First and Second": only "Expertise" (SRD, pp. 13, 39) and "Metamagic" (SRD, p. 44) are mentioned, and if you look at the class tables, it's the same: only the short name is ever written. So I believe that we should check the SRD and keep names as close as possible to it. But the question will rise about the indices: they also still contain the "-choose-" part and this part should be replaced by a single "-". Given that index are kind of elements of a URL, I understand they should be unique, and I believe that we should keep them as unique as possible. And here, I'm inclined to use the 1 and 2 because it just makes sense because it needs to be unique. And for the index, I suggest actually the following format: Where So my suggestion is to have the following example:
There are others, but those examples cover all cases, I guess. What are your thoughts? |
Beta Was this translation helpful? Give feedback.
All reactions
Replies: 3 comments 2 replies
-
I really like that as it mirrors the SRD but is functionally the same.
Beta Was this translation helpful? Give feedback.
All reactions
-
What are your thoughts on how to handle descriptions?
For example, the current description of choose-metamagic is as follows:
"At 3rd level, you gain the ability to twist your spells to suit your needs. You gain two of the following Metamagic options of your choice. You gain another one at 10th and 17th level.", "You can use only one Metamagic option on a spell when you cast it, unless otherwise noted."
Do you think the description should be the same for the choose-additional-mektamagic-1, and so on features?
Regardless, I agree with the changes listed here and would be happy to implement them.
Beta Was this translation helpful? Give feedback.
All reactions
-
Yes, I believe so. We aren't supposed to edit the SRD: only to transcribe it.
Beta Was this translation helpful? Give feedback.
All reactions
-
👍 1
-
Also your current listed indexes for metamagic are incorrect.
They are as follows:
choose-metamagicchoose-additional-metamagic-1choose-additional-metamagic-2...
Should these be changes to metamagic-1, metamagic-2, etc.?
Beta Was this translation helpful? Give feedback.
All reactions
-
I fixed it, sorry. My idea is indeed that we change it to metamagic-1 and friends.
Beta Was this translation helpful? Give feedback.
All reactions
-
👍 2