Conversation
seankross
left a comment
There was a problem hiding this comment.
I LOVE this! Full support, I think the implementation is right. I say ship it.
|
Thanks @seankross - Let me know what you think @tefirman |
|
p.s. I can't ship this as is, as this is just for one chapter. I'll do the rest adding to this PR, then can merge once its all set |
- instead of callouts use _variables.yml file to define rules - and combination of bootstrap badge + text of rule - provides more flexiblity in where rules can be placed (e..g, in lists) - most rules were in lists so it would have been awkward to have callouts combined with lists
|
@seankross can you look at this again (see preview link above)? I changed my mind about how to organize and display rules. callouts were kinda cool but take up a lot of space and have to be on their own lines (can't be in a list, etc.) and most of of our rules are naturally falling within a markdown list of things, so after a bit of trial and error, looks like quarto I've reworked the rules using
Thoughts? |
seankross
left a comment
There was a problem hiding this comment.
I like this too! Full endorsement and full speed ahead.
|
Thanks Sean, merging and we'll continue to iterate from here |

Description
Per our conversations, playing here with using child .qmd files to organize rules that we want to track/use/etc.
Related Issue
#37
Example
I've only done this in one chapter: Package maintenance, open that up and scroll down towards the bottom.
Testing
N/A
Discussion
We could not use callouts too. We could instead do:
Advantages of callouts:
Disadvantages of callouts:
In terms of machine reading, e.g.,
That's just a quick example in R, could do in python too.