-
Notifications
You must be signed in to change notification settings - Fork 14
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
cannot see output #4
Comments
here is
|
Mhm, the html file looks correct. I suspect it could be a browser issue. Did the command open Safari? What happens if you manually open it with another browser (like Firefox or Google Chrome)? |
If you open the browser console, do you see any errors? Another thing to check: in the build folder do you have all the relevant javascript libraries? |
didn't try firefox, but that works! neither safari or chrome did work. |
another way to ask this question: did you give your presentation on your own laptop with firefox installed, or how did you know that would work on computer on site? |
Safari tends to be a tricky case. I gave the talk of firefox, will try to see if I can fix it on Chrome as soon as I have some spare time (browser compatibility really is the most painful part of web development...). |
On a separate note, open issues liberally here: I haven't specified this in the README but this package really didn't get a lot of testing so it's good for me to know whether it also works on someone else computer reliably. |
why can i see this https://piever.github.io/juliacon2018/build/ correclty in safari though? (very nice work by the way!!) |
I've figured it out, chrome doesn't support keeping the markdown in a separate file, I need to put it inside the html. Really not sure why that is but the fix is simple enough... |
Ok, should be fixed on master: could you pull and try again? |
works on both safari and chrome now! good job!thanks! |
hi there,
thanks for making this available. i was trying to follow the example steps
but got a blank browser window:
any ideas? thanks
The text was updated successfully, but these errors were encountered: