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

Update databrowser.html to latest mashlib #640

Merged
merged 2 commits into from Mar 3, 2018
Merged

Conversation

RubenVerborgh
Copy link
Contributor

@RubenVerborgh RubenVerborgh commented Feb 26, 2018

The latest version of the public mashlib at https://linkeddata.github.io/mashlib/dist/mashlib.min.js is no longer compatible with the old mashlib API. As such, databrowser.html has to change.

This PR is a combination of the current databrowser.html at timbl.com and https://github.com/linkeddata/mashlib/blob/c87ca0ac4c71177d29efd4a68ea105650401f0aa/dist/databrowser.html.

@RubenVerborgh
Copy link
Contributor Author

Problem (which also existed with the previous version): document.origin currently only has limited browser support (Chrome only)

Copy link
Contributor

@timbl timbl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Sorry it took to long (where can I find a list of github things waiting for my review?)

@timbl
Copy link
Contributor

timbl commented Mar 3, 2018

Looks good to me and important. Maybe in future we make the databrowser.html file an export of dome sort from mashlib?

@timbl timbl merged commit c311a17 into master Mar 3, 2018
@timbl timbl deleted the fix/databrowser-update branch March 3, 2018 10:47
@RubenVerborgh
Copy link
Contributor Author

(where can I find a list of github things waiting for my review?)

Right here: https://github.com/solid/node-solid-server/pulls/review-requested/timbl

Maybe in future we make the databrowser.html file an export of dome sort from mashlib?

Yes, and/or replace the JavaScript by a single line, such as

Mashlib.init({ pane: 'outliner', subject: location.href, element: 'outline' })

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

Successfully merging this pull request may close these issues.

None yet

2 participants