Skip to content

fix missing unsplash.it background-image for .panel2 #160

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

Merged
merged 1 commit into from
Sep 6, 2017

Conversation

caleywoods
Copy link
Contributor

@caleywoods caleywoods commented Sep 6, 2017

Hey Wes!

Working my way through various lessons and noticed in this flex panels lesson that the image that was previously used for .panel2's background-image is now missing. I've located what is hopefully a suitable replacement and changed it in the two HTML files.

Here is the replacement image suggested in the commit:

https://source.unsplash.com/rFKUFzjPYiQ/1500x1500

@palashmon
Copy link
Collaborator

Yup seems like that image has been removed from the unsplash.it server for some reason. I think we can merge this PR or maybe use static images so that this kind of situation doesn't happen in future.

What's your opinion on this @wesbos?

@wesbos wesbos merged commit cbe8ee7 into wesbos:master Sep 6, 2017
@wesbos
Copy link
Owner

wesbos commented Sep 6, 2017

thank you! Very helpful :)

EmmDguez pushed a commit to EmmDguez/JavaScript30 that referenced this pull request Jun 10, 2018
fix missing unsplash.it background-image for .panel2
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.

3 participants