Skip to content

HW Submission #4 by Crystal Lau - #13

Open
crystalxlau wants to merge 3 commits into
FEWD87:masterfrom
crystalxlau:master
Open

HW Submission #4 by Crystal Lau#13
crystalxlau wants to merge 3 commits into
FEWD87:masterfrom
crystalxlau:master

Conversation

@crystalxlau

Copy link
Copy Markdown

submission

@bmwertman

Copy link
Copy Markdown

@crystalxlau Overall everything looks good but your background images aren't changing for me. I think you only need to;
🔨 Add () to the end of your $("#city-type").val().toUpperCase; so the .toUpperCase method gets called.

@crystalxlau

Copy link
Copy Markdown
Author

@bmwertman Fixed!!

@bmwertman

Copy link
Copy Markdown

Awesome @crystalxlau! That fixed it. Great job! Approved 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants