-
Notifications
You must be signed in to change notification settings - Fork 559
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
這一段話的意思譯反了,請參考新譯 #360
Open
hcchengithub
wants to merge
3
commits into
BonsaiDen:master
Choose a base branch
from
hcchengithub:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
這一段話的意思譯反了,請參考新譯 #360
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,6 +22,7 @@ as objects too. | |
2..toString(); // the second point is correctly recognized | ||
2 .toString(); // note the space left of the dot | ||
(2).toString(); // 2 is evaluated first | ||
2["toString"](); // alternative notation avoids using the dot | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. While this works, I don't think it's necessary |
||
|
||
### Objects as a Data Type | ||
|
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,122 @@ | ||
|
||
[x] The "javascript garden" project needs to be built before browsing according to CONTRIBUTING.md | ||
So I run the build command line: | ||
Microsoft Windows [Version 10.0.14393] | ||
(c) 2016 Microsoft Corporation. All rights reserved. | ||
|
||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm run build | ||
|
||
> [email protected] build c:\Users\hcche\Documents\GitHub\JavaScript-Garden | ||
> node build.js | ||
|
||
module.js:327 | ||
throw err; | ||
^ | ||
|
||
Error: Cannot find module 'lodash' | ||
at Function.Module._resolveFilename (module.js:325:15) | ||
at Function.Module._load (module.js:276:25) | ||
at Module.require (module.js:353:17) | ||
at require (internal/module.js:12:17) | ||
at Object.<anonymous> (c:\Users\hcche\Documents\GitHub\JavaScript-Garden\build.js:2:9) | ||
at Module._compile (module.js:409:26) | ||
at Object.Module._extensions..js (module.js:416:10) | ||
at Module.load (module.js:343:32) | ||
at Function.Module._load (module.js:300:12) | ||
at Function.Module.runMain (module.js:441:10) | ||
|
||
npm ERR! Windows_NT 10.0.14393 | ||
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run | ||
" "build" | ||
npm ERR! node v4.4.3 | ||
npm ERR! npm v2.15.1 | ||
npm ERR! code ELIFECYCLE | ||
npm ERR! [email protected] build: `node build.js` | ||
npm ERR! Exit status 1 | ||
npm ERR! | ||
npm ERR! Failed at the [email protected] build script 'node build.js'. | ||
npm ERR! This is most likely a problem with the javascript-garden package, | ||
npm ERR! not with npm itself. | ||
npm ERR! Tell the author that this fails on your system: | ||
npm ERR! node build.js | ||
npm ERR! You can get information on how to open an issue for this project with: | ||
npm ERR! npm bugs javascript-garden | ||
npm ERR! Or if that isn't available, you can get their info via: | ||
npm ERR! | ||
npm ERR! npm owner ls javascript-garden | ||
npm ERR! There is likely additional logging output above. | ||
|
||
npm ERR! Please include the following file with any support request: | ||
npm ERR! c:\Users\hcche\Documents\GitHub\JavaScript-Garden\npm-debug.log | ||
|
||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden> | ||
[ ] check the error, RI: a module "lodash" is missing. So I studied and found how to install | ||
that module: http://www.w3cplus.com/javascript/lodash-intro.html ( See my Ynote and Evernote ) | ||
--> install lodash: | ||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm install --save lodash | ||
npm WARN package.json [email protected] No repository field. | ||
npm WARN package.json [email protected] No license field. | ||
[email protected] node_modules\lodash | ||
--> ok? Try building j/g again: | ||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm run build | ||
> [email protected] build c:\Users\hcche\Documents\GitHub\JavaScript-Garden | ||
> node build.js | ||
module.js:327 | ||
throw err; | ||
^ | ||
Error: Cannot find module 'jade' | ||
at Function.Module._resolveFilename (module.js:325:15) | ||
at Function.Module._load (module.js:276:25) | ||
at Module.require (module.js:353:17) | ||
at require (internal/module.js:12:17) | ||
at Object.<anonymous> (c:\Users\hcche\Documents\GitHub\JavaScript-Garden\build.js:3:12) | ||
at Module._compile (module.js:409:26) | ||
at Object.Module._extensions..js (module.js:416:10) | ||
at Module.load (module.js:343:32) | ||
at Function.Module._load (module.js:300:12) | ||
at Function.Module.runMain (module.js:441:10) | ||
--> so, again, missing 'jade' module. | ||
--> install jade: | ||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm install --save jade | ||
--> try again: | ||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm run build | ||
Error: Cannot find module 'marked' | ||
at Function.Module._resolveFilename (module.js:325:15) | ||
at Function.Module._load (module.js:276:25) | ||
--> so, again, missing 'marked' module. | ||
--> install marked: | ||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm install --save marked | ||
--> try again: | ||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm run build | ||
|
||
> [email protected] build c:\Users\hcche\Documents\GitHub\JavaScript-Garden | ||
> node build.js | ||
|
||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden> | ||
==> OK now, Bingo! | ||
Taget directory c:\Users\hcche\Documents\GitHub\JavaScript-Garden\site | ||
was very empty but now is full of different languages and index.html appears. | ||
[x] Modify the 'prototype.md' to be : | ||
JavaScript 不包含傳統的「類」繼承的模型,它使用的是「原型」模型。 | ||
儘管這常被認為是 JavaScript 的缺點,但「原型繼承模型」其實比「類繼承模型」更強大。 | ||
在「原型繼承模型」上實現傳統的「類繼承模型」是小事一件,但反過來要在只有「類繼承模 | ||
型」的地方弄出「原型繼承模型」則要困難很多。 | ||
--> submit --> push to my fork. | ||
[x] How to make a PR (Pull request)? | ||
--> Found a [New pull request] button on my repo's home page. --> click it, switched | ||
to a page "Comparing changes" and it says : Choose two branches to see what’s changed or | ||
to start a new pull request. If you need to, you can also compare across forks. --> so | ||
I check the difference and it's fine. The page also says : "Able to merge. These branches | ||
can be automatically merged." so I am about to proceed. | ||
--> And, for the safe, I rebuilt again: | ||
c:\Users\hcche\Documents\GitHub\JavaScript-Garden>npm run build | ||
> [email protected] build c:\Users\hcche\Documents\GitHub\JavaScript-Garden | ||
> node build.js | ||
and then view the changes on my local browser to make sure it works fine: | ||
file:///C:/Users/hcche/Documents/GitHub/JavaScript-Garden/site/zhtw/index.html#object.prototype | ||
--> OK! | ||
--> There's a button "[Create pull request] Discuss and review the changes in this comparison | ||
with others." and I click it . . . | ||
--> follow the process to provide some descriptions and then submit and DONE! | ||
The pull request is really went to the original repo BonsaiDen/JavaScript-Garden | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is jeforth, why is this relevant?