Skip to content
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

improve HTML display #37

Merged
merged 1 commit into from
Oct 2, 2015
Merged

improve HTML display #37

merged 1 commit into from
Oct 2, 2015

Conversation

acoburn
Copy link
Contributor

@acoburn acoburn commented Oct 2, 2015

This updates the rdfs2html.xsl stylesheet to make the HTML output somewhat nicer for models#, use# and rights#

Examples are here:

@azaroth42
Copy link
Contributor

👍

@ruebot
Copy link
Contributor

ruebot commented Oct 2, 2015

Looks great @acoburn!

Only question, concern is the Duraspace logo. Might as well throw that out there for discussion since I'm sure it'll come up. I assume it is there in the absence of a formal PCDM logo.

@azaroth42
Copy link
Contributor

azaroth42 commented Oct 2, 2015 via email

@awoods
Copy link
Member

awoods commented Oct 2, 2015

👍 to this polish. Shall we wait on committing this update until we have a proper PCDM logo?

@daniel-dgi
Copy link

👍 to logo indeed

@daniel-dgi
Copy link

ship it

@ruebot
Copy link
Contributor

ruebot commented Oct 2, 2015

Well, we do have an unofficial spirit animal 😄

hylandora-med

@ruebot
Copy link
Contributor

ruebot commented Oct 2, 2015

...that said, I'm fine with committing as is in the absence of an official logo. Having Duraspace there now is a nice nod to the infrastructure support they have provided in hosting the ontology and letting us use the Duraspace GitHub org.

👍

@scossu
Copy link

scossu commented Oct 2, 2015

👍 for committing now as is. The logo seems to be a separate concern.

@awoods awoods mentioned this pull request Oct 2, 2015
@awoods
Copy link
Member

awoods commented Oct 2, 2015

👍 to committing now.
Follow-on issue created: #38

@ruebot
Copy link
Contributor

ruebot commented Oct 2, 2015

That's 4 committers. I'll merge.

ruebot added a commit that referenced this pull request Oct 2, 2015
@ruebot ruebot merged commit d086316 into duraspace:master Oct 2, 2015
@acoburn acoburn deleted the addFormattingToHTML branch October 2, 2015 19:12
@jpstroop
Copy link

jpstroop commented Oct 2, 2015

Late to the party, but I agree re: a logo.

@awoods
Copy link
Member

awoods commented Oct 5, 2015

Published models updated:
http://pcdm.org/models

@acoburn
Copy link
Contributor Author

acoburn commented Oct 5, 2015

@awoods do you also plan to add the updated rdfs2html.xsl (and assets/ dir) to the http://pcdm.org/use and http://pcdm.org/rights pages?

@awoods
Copy link
Member

awoods commented Oct 5, 2015

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants