-
Notifications
You must be signed in to change notification settings - Fork 21
Christy Elzey - Bootcamp Solutions #75
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
base: master
Are you sure you want to change the base?
Conversation
Week 1.1 HTML Assessments and Week 1.2 CSS Assessments
Fixed for faster transition in 1.2.5
Completed JS Assessments
Completed Assessments for Flexbox and Grid
| <source media="(min-width:1440px)" srcset="https://via.placeholder.com/1600x720"> | ||
| <source media="(min-width:1024px)" srcset="https://via.placeholder.com/1200x540"> | ||
| <source media="(min-width:640px)" srcset="https://via.placeholder.com/800x360"> | ||
| <img src="https://via.placeholder.com/400x180" alt="" style="width:auto"> |
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.
Setting the width to auto makes it not expand/contract to fill the space correctly.
Auto: https://www.screencast.com/t/gISf025v
100%: https://www.screencast.com/t/AuLWjh5UGJX
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.
This should be fixed now. I am not sure why I used 'auto' except it was probably in one of the videos.
| } | ||
| data.set('date', 'October 6, 1986'); | ||
| data.set('time', '21:13 PM'); | ||
| data.set('Powerful, spirited...'); |
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.
This item needs a key of 'mood' like the others have. The value string contains a comma, but that doesn't change the key.
console.log(getInformation('date'));
console.log(getInformation('time'));
console.log(getInformation('mood')); // gives the error message
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 have added the key back into the map.
Fixes for comments on PR
Completed HTML and CSS Assessments