duraspace / pcdm

Portland Common Data Model
http://pcdm.org/models
Apache License 2.0
90 stars 9 forks source link

Create stylesheet for html versions of models, rights, and use. Addre… #24

Closed ruebot closed 9 years ago

ruebot commented 9 years ago

…ss #23.

ruebot commented 9 years ago

@ruthtillman feel free to expand if you think I'm missing anything.

awoods commented 9 years ago

@ruebot, thanks! I was thinking something more along the lines of a stylesheet [1] that was included in the RDF [2] executed in the browser vs. pre-built. [1] https://github.com/fcrepo4/ontology/blob/master/owl2html.xsl [2] https://github.com/fcrepo4/ontology/blob/master/repository.rdf#L2

Thoughts?

ruebot commented 9 years ago

Unless I'm not understanding things right, https://github.com/fcrepo4/ontology/blob/master/repository.rdf#L2 is owl, and PCDM is rdfs. That's why is just tweaked the existing rdfs2html.xsl.

ruebot commented 9 years ago

...oh. I see what you mean now, I think. You mean converting it on the fly?

awoods commented 9 years ago

That is exactly what I mean. Precisely! Just what I mean.

awoods commented 9 years ago

I would like to see "owl:priorVersion" in the actual PCDM ontology documents, ala this, and a corresponding exposure in the HTML view, ala this. But I think this PR, as is, is a very nice improvement. @ruebot, would you be willing to squash these 4 commits prior to another formal review and commit? :+1:

ruthtillman commented 9 years ago

(I know I offered to take a look @ruebot, but it looks really tight from here and I can't even see a way to condense it further. Unsurprising, really.)

ruebot commented 9 years ago

@awoods squashed!

@ruthtillman cool.

awoods commented 9 years ago

:+1: I think this is ready to ship. We need two more votes.

escowles commented 9 years ago

:+1:

ruebot commented 9 years ago

@scossu @azaroth42, thoughts?

awoods commented 9 years ago

We need one more vote: @scossu? @azaroth42?

awoods commented 9 years ago

@scossu? @azaroth42? If you have concerns about this PR, please pass them along. Otherwise, let's get this into master.

azaroth42 commented 9 years ago

:+1: (Sorry for delays)

scossu commented 9 years ago

From my very basic familiarity with XSLT, it looks :+1: