Skip to content

Conversation

@Carreau
Copy link
Member

@Carreau Carreau commented Aug 4, 2012

please backport to 0.13.1

was preventing the display of the static version converted with
nbconvert
@Carreau
Copy link
Member Author

Carreau commented Aug 4, 2012

basically when converted to static with nbconvert everything after the iframe was not renderd.

@takluyver
Copy link
Member

Do iframe tags work if you just end the tag with />?

@Carreau
Copy link
Member Author

Carreau commented Aug 4, 2012

as some content will be injected between opening and closing tag, I wouldn't bet on the ability of the browser to create the closing tag ... and it is not in the spec to allow for <iframe/>

@takluyver
Copy link
Member

OK, I think that's fine, then.

@Carreau
Copy link
Member Author

Carreau commented Aug 4, 2012

you can look at the difference here :
this commit
master constantly up to date

@bfroehle
Copy link
Contributor

bfroehle commented Aug 4, 2012

@takluyver No, iframe is not a void element and therefore must have a closing tag.

Good catch. Merging now.

bfroehle added a commit that referenced this pull request Aug 4, 2012
Fix missing HTML iframe end tags in notebook example.
@bfroehle bfroehle merged commit 079f8ed into ipython:master Aug 4, 2012
minrk added a commit that referenced this pull request Aug 4, 2012
minrk added a commit that referenced this pull request Sep 1, 2012
yarikoptic added a commit to yarikoptic/ipython that referenced this pull request May 2, 2014
* commit 'rel-0.13-33-gcfc5692': (33 commits)
  Backport PR ipython#2347: adjust division error message checking to account for Python 3
  Backport PR ipython#2305: RemoteError._render_traceback_ calls self.render_traceback
  Backport PR ipython#2280: fix SSH passwordless check for OpenSSH
  Backport PR ipython#2270: SSHLauncher tweaks
  Backport PR ipython#2261: Fix: longest_substr([]) -> ''
  Backport PR ipython#2250: fix html in notebook example
  Backport PR ipython#2235: remove spurious print statement from setupbase.py
  fixup
  Backport PR ipython#2223: Custom tracebacks
  Backport PR ipython#2214: use KernelApp.exec_lines/files in IPEngineApp
  Backport PR ipython#2212: catch errors in markdown javascript
  Backport PR ipython#2194: clean nan/inf in json_clean
  Backport PR ipython#2177: remove numpy install from travis/tox scripts
  Backport PR ipython#2169: ipdb: pdef, pdoc, pinfo magics all broken
  Backport PR ipython#2186: removed references to h5py dependence in octave magic documentation
  Backport PR ipython#2185: added test for %store, fixed storemagic
  Backport PR ipython#2170: Fix tab completion with IPython.embed_kernel().
  Backport PR ipython#2163: fix 'remote_profie_dir' typo in SSH launchers
  Backport PR ipython#2117: use explicit url in notebook example
  Backport PR ipython#2126: ipcluster broken with any batch (PBS/LSF/SGE)
  ...
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
Fix missing HTML iframe end tags in notebook example.
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.

3 participants