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

Documentations Issues Continued: can-define/map/map #378

Open
33 of 44 tasks
indifferentghost opened this issue Aug 27, 2018 · 0 comments
Open
33 of 44 tasks

Documentations Issues Continued: can-define/map/map #378

indifferentghost opened this issue Aug 27, 2018 · 0 comments

Comments

@indifferentghost
Copy link
Contributor

indifferentghost commented Aug 27, 2018

This is a continuation of issue #362 separated for can-define/map/map. It was separated to provide amiable and more accurate estimates.

can-define/map/map.html

map/KeysEvent.html

Needs:

  • codepen
  • ES6
  • Syntax Highlighting
  • Use section

Discourse:

  • It's difficult to rationalize what this is for, I'm not sure if this article is a stub or should be tightly linked to something else.

map/PropertyNameEvent.html

Needs:

  • codepen
  • ES6
  • Syntax Highlighting
  • Use section

map/map.extend.html

  • Update to "can" import
  • Add working examples.

Discourse:

  • The examples should probably be moved into separate signatures with further elaboration.

map/map.seal.html

Needs:

  • codepen

Discourse:

  • Does this need a use section? The boolean section seems fairly descriptive.

map/map.prototype.assign.html

Needs:

  • codepen
  • ES6

Discourse:

  • Article is a stub, and doesn't include a Use section. map.assign(props) is fairly descriptive.

map/map.prototype.assignDeep.html

Needs:

  • codepen
  • ES6
  • example actually returns ["first", "2", "3"].

map/map.prototype.deleteKey.html

Needs:

  • codepen
  • ES6
  • Better elaboration that it doesn't delete the keys predefined by type because it would also have to delete those getter and setters on other instances.

Discourse:

  • Article is a stub, and doesn't include a Use section.

map/map.prototype.forEach.html

Needs:

  • codepen
  • ES6
  • Syntax Highlighting

Discourse:

  • While this has a Use section it should probably elaborate further what the example is actually doing.

map/map.prototype.get.html

Needs:

  • codepen
  • ES6

Discourse:

  • map.get(propName) should probably have an example that doesn't include undefined. Might be good to have a verbose example for get and in comparison of how it returns values.
const todo = new DefineMap({ name: 'example', complete: false });
console.log(todo['name'] === todo.get('name')); // true

map/map.prototype.serialize.html

Needs:

  • codepen
  • ES6

Discourse:

map/map.prototype.set.html

Needs:

  • Examples

Discourse:

.get(propName) only should be used when reading properties that might not have been defined yet, but will be later via [can-define/map/map.prototype.set].

  • There should be some elaboration on when to use get vs bracket/dot notation. I've been told the above is incorrect.

map/map.prototype.update.html

Needs:

  • codepen
  • ES6

map/map.prototype.updateDeep.html

Needs:

  • codepen
  • ES6

map/map.prototype.wildcard.html

Needs:

  • Syntax Highlighting
  • ES6
  • codepen
  • reasonable examples for wildcard.

Discourse:

  • It might be worth it to have a verbose use section for all wildcard.
@indifferentghost indifferentghost self-assigned this Aug 28, 2018
indifferentghost added a commit that referenced this issue Aug 29, 2018
…p/KeysEvent.html stating that events.keys.md is a stub.
indifferentghost added a commit that referenced this issue Sep 4, 2018
* updated example to es6 to prototype.assign.md

* Updated example to ES6 syntax. Added link to codepen.

* Updated examples, Added a use section hoping to elaborate what happens when deleteKey is called on a pre-defined property.

* Updated to ES6 syntax. Made codepen-able. Added some documentation to the Use section focused on returning false. Moved the initial example to the signature.

* Updated syntax and and added codepen link.

* updated to ES6. It now links to codepen.

* updateDeep is ES6. It now links to codepen.

* Changed from single to double quote to match other examples.

* added codepen example to the non deprecated use of set.

* get has es6 codepenable examples.

* Syntax, ES6 and link to codepen added. Commented on issue #378 for map/KeysEvent.html stating that events.keys.md is a stub.

* updated events.keys.md spacing

* events.propertyName.md updated example.

* updated syntax and linked to codepen. Also threw them down to a use section, though it might be suggested to change them to separate signatures.

* fixed spacing error in static.extend.md

* major updates to define-map.md documentation.

* added syntax highlighting and updated to es6 syntax for wildcard.

* working es6 codepen-able example on static.seal.md

* minor cleanup

* some additional fixes

* use any now for accepting any type

* updated assign and assignDeep to more mirror their counterparts in list/list.

* codepen updates across map.

* Updated and fixed spelling errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants