-
Notifications
You must be signed in to change notification settings - Fork 19
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
[DISC] Add {proof}
as new role
#13
Comments
Just my two cents, but I wish {eq} was {eqref}, analogous to Hence I favor {proof:ref} over {proof}. For the other suggestions, is this in response to some unmet need? I like the philosophy of @choldgraf --- wait until people request features and then solicit PRs or add them as requested. |
I am also in favour of Can you give an example re: placeholders? |
Here are three examples of how one could use placeholders when referencing labels through
The I'm more than happy to follow @jstac's suggestion and wait until there's a need for supporting the placeholders. |
oh I see -- that's a nice idea. |
I think we should add
{proof}
as a new role to "replace" the current role which is{proof:ref}
. Technically, the two would be interchangeable, similarly to how{math:numref}
and{eq}
roles are interchangeable.In addition to this, I think we should also support the following placeholders:
%s
,{number}
: int or '' ifnonumber == True
{title}
: str or '' if no title was providedWhat do others think? @mmcky @AakashGfude @jstac
The text was updated successfully, but these errors were encountered: