Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[javascript/en]: add es6 updates to javascript. fixes #939 #1905

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
7 participants
@srrvnn
Copy link
Contributor

srrvnn commented Oct 30, 2015

added introduction to es6. and the following concepts:

assignment destructuring / spread / rest / arrow functions / template literals / object literals / classes / let and const / symbols / iterators / maps / sets / proxies / modules.

added link to babeljs.io and esfiddle.net

@adambrenecki

This comment has been minimized.

Copy link
Collaborator

adambrenecki commented Oct 31, 2015

This all looks really good, but I wonder if it's a bit too long - it adds a whole 40% to the length of the file!

IMO, the iterator API is way too nuts and bolts for this sort of document (although I do think for ... of should be in there, maybe with a 1-sentence explanation of how it's different to for ... in). That's a fair chunk of it. I'm in two minds how essential symbols and proxies are, too; IMO other than that the rest is all stuff that definitely needs to be there.

@vendethiel

This comment has been minimized.

Copy link
Collaborator

vendethiel commented Jun 26, 2016

@srrvnn sorry for the delay, it seems that has merge conflicts now.

@vendethiel

This comment has been minimized.

Copy link
Collaborator

vendethiel commented Jun 26, 2016

Actually... This might want to be its own article. @adambard ?

@geoffliu

This comment has been minimized.

Copy link
Collaborator

geoffliu commented Aug 21, 2016

@vendethiel @adambard

Just my $0.02:

Personally I don't think it's worth maintaining two articles, since the Javascript community has been quick to adapt to the new standard. With the myriad transpilers and polyfills out there, I see little point in keeping the old syntax around.

I propose we add a link to the article that points to the current git commit, so if people really want, they can go find the old syntax (something like this: https://github.com/adambard/learnxinyminutes-docs/blob/6e9e1f4af018dc0e277c968d9b5fe011e0a1ce97/javascript.html.markdown). Going forward, we can update the syntax to the new ES6 standard.

@vendethiel

This comment has been minimized.

Copy link
Collaborator

vendethiel commented Aug 21, 2016

Fair enough.

@vendethiel

This comment has been minimized.

Copy link
Collaborator

vendethiel commented Oct 21, 2016

Still unsure what to do with this one and #1844.

@adambard

This comment has been minimized.

Copy link
Owner

adambard commented Oct 21, 2016

Experienced javascript devs may be using ES6 with transpilers, but I suspect a majority of people looking for a JS syntax reference are probably going to want stuff that will work in browsers unmodified, and there's still a lot of red in the compatibility table. I'd be ok with an ES6 addendum, but for the moment I still think limbo is the appropriate place for this PR.

@geoffliu geoffliu referenced this pull request Oct 31, 2016

Closed

Add ES6 information to JavaScript file #2491

4 of 4 tasks complete
@samcv

This comment has been minimized.

Copy link
Collaborator

samcv commented Nov 11, 2016

I think either putting all the ES6 stuff at the bottom in its own section would be good as long as it's clear that it's not all supported yet in browsers.

@vendethiel

This comment has been minimized.

Copy link
Collaborator

vendethiel commented May 25, 2017

Hey!

I think it's about time we start adding ES6 info.
There's another PR for it: #2491.
I'm not sure whichever should be picked, but if we could get them "merged" I think it'd be pretty great.
If you have some time to look at the other PR, please tell us!

@adambrenecki

This comment has been minimized.

Copy link
Collaborator

adambrenecki commented May 26, 2017

I had the same thought not long ago—most browsers support nearly all of ES6, so it's probably time to add it to the LXIYM (with explanatory notes).

My intention is to combine and edit the two existing PRs myself, but I probably won't get to that until next weekend (I've got a Uni assignment to finish and a meetup talk to prepare this weekend 😦).

@adambrenecki adambrenecki referenced this pull request Sep 17, 2017

Open

[javascript/en][WIP] ES2015 #2858

9 of 18 tasks complete
@divayprakash

This comment has been minimized.

Copy link
Collaborator

divayprakash commented Nov 1, 2018

Closed in favor of #2858.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.