Conversation
Still trying to fix format issue.
Suggest moving problematic backticks to the end of the relevant lines.
|
This is a weird one. I think the problem is the mixture of indented lists and fenced code blocks. @nataled I made a commit that does something different but looks ok to me. What do you think? |
|
@jamesaoverton if the HTML looks like the 'preview', then I do not like it. It removes a newline that, when missing, makes it more difficult to read. |
| ``` | ||
| consider: OBI:0001544 | ||
| ``` | ||
| - OWL: Use the `oboInOwl:consider` annotation property with the value set to the full IRI(s) of the relevant term(s): `<oboInOwl:consider rdf:resource="http://purl.obolibrary.org/obo/OBI_0001544">` |
There was a problem hiding this comment.
I'd like to first try the change I originally made. This change will make it harder to read.
Try again with more blank lines
nlharris
left a comment
nlharris
left a comment
There was a problem hiding this comment.
I am not sure how to review this until it is merged and I can see the final formatting.
|
Ok, well here's another version, but it's impossible to get the list level for the first and second items to match if you do it this way: The fenced code block is a top-level element that will reset your list. |
|
@nlharris I think we'll have to bite the bullet and approve so that we can see what changed, for better or worse, and take it from there. |
|
I tested my changes locally, starting with the instructions here: https://github.com/OBOFoundry/OBOFoundry.github.io/blob/master/README-sitedev.md. They're fairly technical, and didn't work perfectly for me anyway. We could consider using a preview tool like this one https://github.com/marketplace/actions/deploy-pr-preview but I'm not volunteering to set up and maintain that. |
|
Whatever change just made DID fix the main problem. Just one more to deal with... |
3 participants
Still trying to fix format issue.