-
Notifications
You must be signed in to change notification settings - Fork 15
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
Homework submission from Anastasia Schultis #12
base: master
Are you sure you want to change the base?
Conversation
js/app.js
Outdated
|
||
// Create a function for the Submit buttom | ||
function changeCity() { | ||
console.log("clicked on Submit button") |
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.
@Schultisal remove console.log 🔨
js/app.js
Outdated
|
||
// use else or else if when you type in city name | ||
// specify lowercase/uppercase for the cities | ||
// if() { |
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.
@Schultisal remove commented code 🔨
js/app.js
Outdated
// } else if() { | ||
|
||
if (city === "New York" || city === "NYC" || city === "New York City") { | ||
$('body').css('background-image', "url('images/nyc.jpg')"); |
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.
@Schultisal the is an extra "
next to url
🔨
js/app.js
Outdated
|
||
if (city === "New York" || city === "NYC" || city === "New York City") { | ||
$('body').css('background-image', "url('images/nyc.jpg')"); | ||
console.log("changed to NYC"); |
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.
@Schultisal remove console.log 🔨
js/app.js
Outdated
|
||
else (city === "San Francisco" || city === "SF") { | ||
$('body').css('background-image',"url('images/sf.jpg'); | ||
console.log("Changed to SF"); |
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.
@Schultisal remove the console.log 🔨
@Schultisal good work on the assignment, just make a note of the comments |
all the pictures work for UpperCase
No description provided.