Skip to content

Week 1 Homework#32

Open
janauhrich wants to merge 2 commits into
modern-web-application-uw19:masterfrom
janauhrich:master
Open

Week 1 Homework#32
janauhrich wants to merge 2 commits into
modern-web-application-uw19:masterfrom
janauhrich:master

Conversation

@janauhrich
Copy link
Copy Markdown

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): 3
  • Completion rating on this assignment (complete/incomplete): complete
  • Github handles of code I've reviewed: n/a

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

Choose a reason for hiding this comment

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

Try to take advantage of the parent constructor, and pass the variables directly into super since it already handles variable assignment.

Comment thread src/timer/Timer.js
}

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.

👍

@MadEste
Copy link
Copy Markdown

MadEste commented Apr 27, 2019

👍

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