-
Notifications
You must be signed in to change notification settings - Fork 61
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
🔢 Add enumeration options to common directives #1477
Conversation
🦋 Changeset detectedLatest commit: a165d36 The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the refactoring of options for directives. We should extend this to class
in the future.
Co-authored-by: Angus Hollands <[email protected]>
Yes - I purposely left |
Co-authored-by: Angus Hollands <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good - relies on the fact that we already respect enumerated
flag. I like the shared option refactor.
}, | ||
...labelDirectiveOption('blockquote'), | ||
|
||
// TODO: Add enumeration in future |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good that we are not adding enumerator
on places where we do not number stuff yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking forward to being able to number everything ... !
This adds
enumerated
/enumerator
(aliased asnumbered
/number
) to all common directives (code, figures, etc.). This allows for things like turning off enumeration for a specific figure / equation as well as taking control of numbering completely.In a recent example, we had to create a figure with a specific number
S5
and then carry on numbering as usual.This also introduces some places to centralize directive option logic - which we could extend to classes in the future.