Collapsible Headings breaks the shift-tab tooltip in Jupyter Notebook 5.0.0

[jbwhit]

As detailed in this issue: jupyter/notebook#2383

Here is what it looks like on my setup:

And here's the output from jupyter nbextension list:

$ jupyter nbextension list
Known nbextensions:
  config dir: /Users/jonathan/.jupyter/nbconfig
    notebook section
      collapsible_headings/main  enabled
      - Validating: OK
      codefolding/main  enabled
      - Validating: OK
      toc2/main disabled
      nbextensions_configurator/config_menu/main  enabled
      - Validating: problems found:
        - require?  X nbextensions_configurator/config_menu/main
  config dir: /Users/jonathan/miniconda3/envs/py3/etc/jupyter/nbconfig
    notebook section
      jupyter-js-widgets/extension  enabled
      - Validating: OK
    tree section
      nb_conda/tree  enabled
      - Validating: OK
  config dir: /usr/local/etc/jupyter/nbconfig
    tree section
      ipyparallel/main  enabled
      - Validating: OK

Other comments are stating that Collapsible Headings is where the break is coming from.

[jcb91]

Thanks for the note. So this is presumably related to a patch applied to Tooltip._show by collapsible_headings in order to deal with making cells position:relative. My guess is that the tooltip mechanism changes in 5.0.0, but the patched version is still working in the old way. I'll try to look at this in the coming few days...

[jcb91]

Presumably related to jupyter/notebook@5945873

[jcb91]

yep, it seems that in 5.0.0 the patch is not needed, since the cell position is already relative rather than static as it used to be in 4.x. I've added PR #950, which should hopefully fix this. If you get a chance to try it & could let me know how it works for you, that'd be great 😄

[wikiped]

I think return; here breaks the Promises chain processing and the whole extension doesn't work with Jupyter 5.x.

I tried return Promise.resolve(); and this seems to solve the issue and keeps extension functioning as expected.

[jcb91]

d'oh, good point, I was in a rush to finish this this morning and didn't test this properly...

[jcb91]

ok, updated with the promise fix, so given you say that works for you, and it seems to work for me on 4.x, I'll go ahead and merge it soon, I think

[jcb91]

fixed in #950