Skip to content

week1 hw#16

Open
ryancfields wants to merge 1 commit into
frontend-application-development-uw20:masterfrom
JS300A:week1_hw
Open

week1 hw#16
ryancfields wants to merge 1 commit into
frontend-application-development-uw20:masterfrom
JS300A:week1_hw

Conversation

@ryancfields
Copy link
Copy Markdown

@ryancfields ryancfields commented Apr 15, 2020

Week 1 HW Submission

Please fill out the information below in order to complete your assignment. Feel free to update this comment later if necessary.

  • Comfort rating on this assignment (1-5): 5
  • Completion rating on this assignment: complete

Comment thread package.json
"main": "index.js",
"scripts": {
"lint": "eslint ./src",
"lint": "eslint --fix ./src",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😄

// Define class here
class DivElement extends HTMLElement {
constructor(content) {
super('div', content);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

const phone = person.phone;

console.log('First name: ' + firstName + '\nLast name: ' + lastName + '\nEmail: ' + email + '\nPhone number: ' + phone + '\n');
people.forEach((element) => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
people.forEach((element) => {
people.forEach(({ name, phone, email }) => {


console.log('First name: ' + firstName + '\nLast name: ' + lastName + '\nEmail: ' + email + '\nPhone number: ' + phone + '\n');
people.forEach((element) => {
const [firstName, lastName] = element.name.split(' ');
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
const [firstName, lastName] = element.name.split(' ');
const [firstName, lastName] = name.split(' ');

Comment thread src/timer/Timer.js
}, 1000);
};
start() {
const timerInterval = setInterval(() => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work!

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.

2 participants