-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: Allow custom, per-card, cloze context value #97
base: develop
Are you sure you want to change the base?
Conversation
f167ac8
to
142805e
Compare
Wow, you're fast. This is pretty much exactly what I had in mind. As I understand it, the new property is set each time a cloze card is initialized / updated For this reason, I'd prefer a function that optionally adds the property, prompting for a numeric value. |
142805e
to
da67dd9
Compare
The trick is to forget to create the issue until you're more than half-way through implementing the feature.
Done: |
Add classes for cards and positions
fix: Use class accessor in org-fc-cache
837dc25
to
335f190
Compare
Checking in -- is there anything you'd like to see changed in this pull request? |
It just occurred to me that there is already a mechanism for this in the form of the Two missing parts are a setter function (probably to hard to generate using the macro, due to the interactive-read) Despite this, I think the macro-based approach will avoid some code duplication once more variables of this kind are added. |
Reference: #96