Skip to content

Stephanie Banks CS12 - #164

Open
steviebanko wants to merge 4 commits into
bloominstituteoftechnology:masterfrom
steviebanko:master
Open

Stephanie Banks CS12#164
steviebanko wants to merge 4 commits into
bloominstituteoftechnology:masterfrom
steviebanko:master

Conversation

@steviebanko

Copy link
Copy Markdown

No description provided.

Comment thread assignments/closure.js
// ==== Challenge 1: Write your own closure ====
// Write a simple closure of your own creation. Keep it simple!

function styleFont(size, /*color*/){

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Like we talked about, a more explicit and easier explanation will typically involve two functions so there is a very clear use and need for the closure. Just keep this in mind as closure is a very common interview topic.

@Nate152

Nate152 commented Jun 27, 2018

Copy link
Copy Markdown

Awesome job Steph. This looks great. Try to be consistent with your use of either arrow functions or normal function syntax. Its not a huge deal in this particular case but we want to build good habits and a good habit is to be consistent with things of this nature. Another good example would be single versus double quotes. You can use either, you just need to be consistent whichever you choose. A little bit more that we can come back to and work on but I love to see that you put in some serious work here. Be proud and keep plugging away.

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