Skip to content

0.8 property config - #1256

Merged
kevinpschaaf merged 10 commits into
0.8-previewfrom
0.8-property-config
Mar 5, 2015
Merged

kevinpschaaf merged 10 commits into
0.8-previewfrom
0.8-property-config

Conversation

@kevinpschaaf

Copy link
Copy Markdown
Member
  • Renames published configuration object to properties
    • Adds computed field to properties. Replaces use of general computed object
    • Adds observer field to properties. Replaces use of general bind block for single-property observers
    • Adds value field to properties. Each property in properties accepts a value field to define a default value; may either be a static value or function that returns a value (e.g. for initializing objects/arrays); replaces use of general configure function (although general function still exists for exotic use cases)
  • Removes general computed configuration object (these must move to properties per property)
  • Renames bind to observers (general observers configuration object remains for path and multiple-dependency observers; will still work for single-property observers, although these should generally go in properties)
  • Adds multiple-dependency observer support to general observers object
  • Fixes attribute deserialization loop when setting reflect: true properties
  • Adds attribute deserialization tests

Minimal element changes required for compatibility with this API update:

  • Rename published to properties
  • Move all computed properties into properties object
  • Rename bind to observers

Not strictly required, but should update to match style going forward:

  • Move all single-property observers (formerly bind) into observer field of properties object
  • Move all default values defined in configure to value field of properties object (and remove configure function)

Comment thread property-config.html Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

delete this file.

@sorvell

sorvell commented Mar 5, 2015

Copy link
Copy Markdown
Contributor

LGTM with minor comments.

kevinpschaaf added a commit that referenced this pull request Mar 5, 2015
@kevinpschaaf
kevinpschaaf merged commit 42477e4 into 0.8-preview Mar 5, 2015
@kevinpschaaf
kevinpschaaf deleted the 0.8-property-config branch March 5, 2015 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants