Submitting Homework 2 #7

Merged
merged 2 commits into from Dec 12, 2018

Conversation

Projects
None yet
2 participants
@cubofantastico
Contributor

cubofantastico commented Dec 3, 2018

No description provided.

@vodnik

vodnik approved these changes Dec 12, 2018

// function parameter?)

for (let i = 1; i <= 100; i++) {
if (((i % 3) + (i % 5)) === 0) {

This comment has been minimized.

@vodnik

vodnik Dec 12, 2018

Owner

Nice use of nested parens to convey your meaning.

breed:'Beagle',
noise:'Bark',
makeNoise: function() {
console.log(this.noise);

This comment has been minimized.

@vodnik

vodnik Dec 12, 2018

Owner

Nice work using this to keep your object flexible.

BONUS 3: Clear the contents of the text box after you've displayed your results, so a user can easily enter another temperature to convert. (Hint: Read about the 'value' property in JavaScript. What string value could you assign as the value of an input box that would give it no content to display?)
*/

function toCelcius(startTemp) {

This comment has been minimized.

@vodnik

vodnik Dec 12, 2018

Owner

Short and sweet. Nicely done!

// added.)

const myImage = document.createElement('img');
myImage.src = 'images/about.jpg';

This comment has been minimized.

@vodnik

vodnik Dec 12, 2018

Owner

Great job building out an img element and src element.

// console.log(link);
// }

for (let i of flickrObj.items) {

This comment has been minimized.

@vodnik

vodnik Dec 12, 2018

Owner

Well done converting a for loop to a for/of statement!

@vodnik vodnik merged commit e7f6d72 into vodnik:master Dec 12, 2018

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