Skip to content

Conversation

@RubenVerborgh
Copy link
Contributor

@RubenVerborghRubenVerborgh 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
ContributorAuthor

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

Copy link
Contributor

@timbltimbl 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?

@timbltimbl merged commit c311a17 into masterMar 3, 2018
@timbltimbl deleted the fix/databrowser-update branch March 3, 2018 10:47
@RubenVerborgh
Copy link
ContributorAuthor

(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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@RubenVerborgh@timbl