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

UI doesn't render if custom.js missing #32

Closed
takluyver opened this issue Apr 19, 2015 · 5 comments
Closed

UI doesn't render if custom.js missing #32

takluyver opened this issue Apr 19, 2015 · 5 comments

Comments

@takluyver
Copy link
Member

We load custom/custom.js using require along with all of our own modules, and page.show() is called in the success callback for that require call. So if there's a 404 on custom.js, you're left with a blank page, and only a message at the JS console says what the problem is.

We should load custom.js separately, since it doesn't matter if it's not there.

It would also be good to have something more informative than a blank page if something fails to load, but that's not the focus of this issue.

@takluyver takluyver added the bug label Apr 19, 2015
@takluyver takluyver added this to the 4.0 milestone Apr 19, 2015
@minrk
Copy link
Member

minrk commented Apr 19, 2015

It shouldn't be possible for custom.js to be missing. This is a bug introduced by #16.

@minrk
Copy link
Member

minrk commented Apr 19, 2015

Actually, this doesn't appear to be true. I ran master without any custom.js, and it loads fine.

@takluyver
Copy link
Member Author

Well, I ran master and it didn't load, but when I commented out the 'custom/custom' line in main.js, it worked. Maybe it was cached when you tried?

@minrk
Copy link
Member

minrk commented Apr 19, 2015

I can't seem to make it 404. Did you restart your nbserver? #16 changed the URL for custom from /static/custom to /custom since it's handled specially. What do you see at http://localhost:8888/custom/custom.js?

@takluyver
Copy link
Member Author

Ah, I must have pulled while I had a notebook server running. Sorry about that.

@takluyver takluyver modified the milestones: no action, 4.0 Apr 19, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 4, 2021
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants