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

Fix break in Plotting in Python Using Matplotlib and some CSS fixes #97

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

jmuzsik
Copy link

@jmuzsik jmuzsik commented Jan 14, 2020

👋 Hi

This is a simple fix. I had an error when visiting https://nbviewer.jupyter.org/github/qutip/qutip-notebooks/blob/master/python/Matplotlib_Plotting.ipynb. As, the code examples were hidden. The reason stated in the console was:

Mixed Content: The page at 'https://nbviewer.jupyter.org/github/qutip/qutip-notebooks/blob/master/python/Matplotlib_Plotting.ipynb was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Source+Sans+Pro'. This request has been blocked; the content must be served over HTTPS.`

So, I replaced the http://fonts.googleapis.com with https://fonts.googleapis.com and that should do the trick (it cannot be checked on localhost). I also ran an automatic linter on the css code and deleted a couple errors.

@nathanshammah nathanshammah self-requested a review May 26, 2020 10:44
@nathanshammah
Copy link
Member

Hi @jmuzsik, thank you for the feature addition. I am no html expert, but in that notebook it says:
"The raw code for this IPython notebook is by default hidden for easier reading. To toggle on/off the raw code, click here."
It seems to work fine for me, on Chrome, Safari and Firefox.

Copy link
Member

@nathanshammah nathanshammah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would only switching to https be fine?

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

Successfully merging this pull request may close these issues.

2 participants