Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

Compatibility update with widgets for Safari / Firefox support #33

Merged
merged 3 commits into from
Oct 14, 2015

Conversation

parente
Copy link
Member

@parente parente commented Oct 14, 2015

To fix Safari and Firefox support in declarative widgets (jupyter/declarativewidgets#21) the path passed to the widget init function needed to change (jupyter/declarativewidgets#28).

This PR contains the path change in the dashboard main.js and a test notebook. It should not be merged until after the PR above is merged for widgets.

(c) Copyright IBM Corp. 2015
* Everything must have is=“ruth-core-import” now
* urth_components path must be relative

(c) Copyright IBM Corp. 2015
@parente parente changed the title Update widget init path Compatibility update with widgets for Safari / Firefox support Oct 14, 2015
@parente
Copy link
Member Author

parente commented Oct 14, 2015

Pushed a third commit correcting our demo notebooks for new urth widget requirements:

  1. hrefs must start with `urth_components/`
  2. is="urth-core-import" must appear on all widget link elements, including urth-* ones

/cc @jhpedemonte @dalogsdon @Lull3rSkat3r: the widget change will impact your notebooks in progress

@lbustelo lbustelo merged commit bcdd512 into jupyter:master Oct 14, 2015
@lbustelo
Copy link
Contributor

Ran through all the modes and browsers and it the import issue is fixed.

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

Successfully merging this pull request may close these issues.

2 participants