Repository navigation
Conversation
11b2a77 to
8ab8207
Compare
8ab8207 to
1a7f15f
Compare
There was a problem hiding this comment.
I can't leave comments by the lines of the files in the submodules.
2 tables in 5_structure_package.md are broken due to the hidden header in the wrong place:
mets/dmdSec/mdRef
mets[@csip:CONTENTINFORMATIONTYPE="OTHER"]/@csip:OTHERCONTENTINFORMATIONTYPE
And 1 due to a missing column in the delimiter row.
Overview of relevant PREMIS relationships
The latter issue is also present in 6_structure_representation.md and both versions of film.md
There was a problem hiding this comment.
Other broken table (inadvertently split a table) in **/6_*.md:
elements-and-internal-references
|
Just thinking out loud. These headerless tables look more like description lists to me. Apparently there's a markdown extension that can be enabled: Syntax would be: These show up on separate lines, but could be styled using CSS to show up in 2 columns. Might be more in line with semantic web sensibilities? |
|
@jaameemoo you are absolutely right. Deflists are definitely a better fit. I will have a look at my script to migrate those. |
|
@jaameemoo I fixed the tables and made the switch. Could you do another check? |
mnkdocs does not support headerless tables, so this PR adds an empty header and hides it.