Skip to content

Conversation

@siddharthkp
Copy link
Contributor

Ref: #261

(only for english)

Foo.bar = 1;
Foo.bar; // 1
function sayHello(){}
sayHello.bar = 1;
Copy link
Collaborator

Choose a reason for hiding this comment

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

you missed a 'bar' :)

The properties of an object can be accessed in two ways, via either the dot
notation or the square bracket notation.

var foo = {name: 'kitten'}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Haha - this is a great change. I wonder y someone decided on 'kitten' and stuck with foo as the variable name :D

operator; setting the property to `undefined` or `null` only removes the
*value* associated with the property, but not the *key*.

var obj = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm less keen on this change. If we're not replacing 'foo' with something meaningful & realisitic, I think foo, bar, baz is actually preferable to 'a','b','c'. More people will realise it's an explicit 'name does not matter' placeholder.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, that was the exact reason in my head to make it abc.

Because of this comment: #261 (comment)

foo and bar are specially confusing for non english speakers (like me) because we can't translate/understand these words

foo, bar, etc. might seem like they're supposed to make sense. abc are alphabets as placeholder 😅

@timruffles timruffles merged commit 5fb802c into BonsaiDen:master Oct 9, 2017
@timruffles
Copy link
Collaborator

Looks great, thanks! Would it be possible to take a look at the comments and make another PR too?

@siddharthkp siddharthkp mentioned this pull request Oct 9, 2017
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