Skip to content

Change the way GetElementAttribute handles attribute names#123

Merged
skyline-gleb merged 1 commit intomasterfrom
change-get-attribute
Dec 28, 2015
Merged

Change the way GetElementAttribute handles attribute names#123
skyline-gleb merged 1 commit intomasterfrom
change-get-attribute

Conversation

@NickAb
Copy link
Copy Markdown
Contributor

@NickAb NickAb commented Dec 17, 2015

This breaks v1.5.0 GetElementAttribute compatibility

To access Automation Properties the name of property:

  • must prefixed by AutomationProperties.
  • must not be suffixed by Property
  • examples:
    • AutomationProperties.AutomationId
    • AutomationProperties.Name

To access Dependency Properties the name of the property:

  • must not be suffixed by Property
  • examples:
    • IsReadOnly

This breaks v1.5.0 GetElementAttribute  compatibility

To access Automation Properties the name of property:
 - must prefixed by `AutomationProperties.`
 - must not be suffixed by `Property`
 - examples:
  - `AutomationProperties.AutomationId`
  - `AutomationProperties.Name`

To access Dependency Properties the name of the property:
 - must not be suffixed by `Property`
 - examples:
  - `IsReadOnly`
@NickAb NickAb changed the title Change the way GetElementAttribute handles attribute names [WIP] Change the way GetElementAttribute handles attribute names Dec 17, 2015
@NickAb
Copy link
Copy Markdown
Contributor Author

NickAb commented Dec 17, 2015

Fixes #122

@NickAb NickAb changed the title [WIP] Change the way GetElementAttribute handles attribute names Change the way GetElementAttribute handles attribute names Dec 28, 2015
skyline-gleb added a commit that referenced this pull request Dec 28, 2015
Change the way GetElementAttribute handles attribute names
@skyline-gleb skyline-gleb merged commit 824e669 into master Dec 28, 2015
@skyline-gleb skyline-gleb deleted the change-get-attribute branch December 28, 2015 05:47
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