-
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
Add funding logos #80
Conversation
✅ Deploy Preview for ngff-challenge ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
<table> | ||
<tr> | ||
<td><img src="nfdi_rgb_Wortmarke_Zusatz_quer.png" width="45%"/></td> | ||
<td><img src="czi-logo-chan-zuckerberg-initiative-logo.png" width="45%"/></td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was going to say that this won't work when deployed on github, and suggest that you need to do it as in Nav.svelte:
import omeMainNav from "/ome-main-nav.svg";
...
<img alt="OME Logo" src={omeMainNav} />
But I just pushed this branch to my 'main' branch so it's deployed at https://will-moore.github.io/ome2024-ngff-challenge/about/ and it's fine!
The approach I used for Nav is possibly a bit more robust since it generates an absolute URL, but actually this is working fine so 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks to be working. Thanks.
No description provided.