Skip to content

week 1 assignment#24

Open
zzxjoanw wants to merge 8 commits into
modern-web-application-uw19:masterfrom
zzxjoanw:master
Open

week 1 assignment#24
zzxjoanw wants to merge 8 commits into
modern-web-application-uw19:masterfrom
zzxjoanw:master

Conversation

@zzxjoanw
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:

class DivElement extends HTMLElement {
constructor(content) {
super('div', content);
this.content = 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.

Unnecessary since parent constructor already handles content assignment.

@MadEste
Copy link
Copy Markdown

MadEste commented Apr 26, 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