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

missing image on load page #12

Closed
Labels
good first issue Good for newcomers

Comments

@rabernat
Copy link
Member

I assume there should be a pangeo logo at the top of this page, no?

image

@jhamman
Copy link
Member

jhamman commented Sep 30, 2018

Yes, we should fix this. I'm not sure why this isn't working. I think this page should be inhereting from the 'page.html' but something does seem to be broken.

https://github.com/pangeo-data/pangeo-custom-binderhub-templates/blob/0216d32700742c5c31204fd8be0198f25eba10d2/templates/page.html#L18-L28

I'd bet @dsludwig knows what's up here.

@rabernat
Copy link
Member Author

If anyone wants to fix this in the next 10 minutes that would be great. 😜

@minrk
Copy link
Contributor

minrk commented Nov 8, 2018

It's due to the url prefix being a relative URL, which evaluates differently on different pages. #16 should fix it.

@jhamman jhamman reopened this Nov 9, 2018
@jhamman
Copy link
Member

jhamman commented Nov 9, 2018

Hmmm, this still seems to be a problem. Will look into it more.

@jhamman jhamman added the good first issue Good for newcomers label Dec 5, 2018
@andersy005
Copy link
Member

This appears to be now fixed.

@jhamman
Copy link
Member

jhamman commented Dec 5, 2018

I'm still seeing:
image

@andersy005
Copy link
Member

You are right. I was looking at the wrong page

@rabernat
Copy link
Member Author

rabernat commented Dec 5, 2018

This is the URL we are looking for. It exists
https://binder.pangeo.io/extra_static/pangeo_simple_logo.svg

The logo link is rending as

<img id="logo" src="extra_static/pangeo_simple_logo.svg" width="390px"  />

with a relative, rather than absolute, URL.

The problem is with these lines:

template:
path: "/etc/binderhub/templates/templates"
static:
path: "/etc/binderhub/templates/static"
urlPrefix: /extra_static/

@minrk added the extra / to url prefix. But it didn't work. Does it need to be in quotes perhaps?

@jhamman
Copy link
Member

jhamman commented Dec 7, 2018

@rabernat - I just tried with your attempt -- no joy!. Now the image is coming from: https://staging.binder.pangeo.io/v2/gh/jhamman/dask-examples/extra_static/pangeo_simple_logo.svg

So we need to strip out the repo info: https://staging.binder.pangeo.io/extra_static/pangeo_simple_logo.svg

@bitnik
Copy link

bitnik commented Dec 7, 2018

@jhamman @rabernat I think the problem is in page.html: https://github.com/pangeo-data/pangeo-custom-binderhub-templates/blob/master/templates/page.html#L22

Leading / is missing. It should be like:

        <img id="logo" src={% block logo_image %}"/extra_static/pangeo_simple_logo.svg"{% endblock logo_image %} width="390px"  />

pangeo-data/pangeo-custom-binderhub-templates#2 didn't fix the logo in loading page because loading page uses loading.html not index.html.

I would also delete logo_image block in index.html (https://github.com/pangeo-data/pangeo-custom-binderhub-templates/blob/master/templates/index.html#L10), since index.html is extending page.html and they have the same logo.

@jhamman
Copy link
Member

jhamman commented Dec 7, 2018

Thanks @bitnik. That gets us closer. Any idea why the change in pangeo-data/pangeo-custom-binderhub-templates@17bfade would cause the page header css to be stripped out?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
5 participants