-
Notifications
You must be signed in to change notification settings - Fork 10
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
Draft - Framework integration #1980
Draft - Framework integration #1980
Conversation
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.
Great start, this is very useful! I left a few questions about missing sections, apart from that most comments are small nitpicks and suggestions
docs/001_develop/03_client-capabilities/022_framework-integration/index.mdx
Outdated
Show resolved
Hide resolved
docs/001_develop/03_client-capabilities/022_framework-integration/index.mdx
Outdated
Show resolved
Hide resolved
docs/001_develop/03_client-capabilities/022_framework-integration/index.mdx
Outdated
Show resolved
Hide resolved
docs/001_develop/03_client-capabilities/022_framework-integration/002_integration-angular.mdx
Outdated
Show resolved
Hide resolved
|
||
Attributes are part of the HTML markup of an element and are reflected in the dom. | ||
|
||
Attributes can be either hard coded or set using Angular's property event bindings. Note, any automatically maps hyphenated attribute names to camelCase property names so a property called `max-rows` becomes `maxRows`. |
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.
"Note, any automatically maps hyphenated attribute names to camelCase property" this sentence doesn't sound correct
docs/001_develop/03_client-capabilities/022_framework-integration/002_integration-angular.mdx
Show resolved
Hide resolved
docs/001_develop/03_client-capabilities/022_framework-integration/003_integration-react.mdx
Outdated
Show resolved
Hide resolved
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.
Basically the same comments I left on the Angular page apply here too
…ion/index.mdx Co-authored-by: Matt Walker <43502076+matteematt@users.noreply.github.com>
…ion/index.mdx Co-authored-by: Matt Walker <43502076+matteematt@users.noreply.github.com>
…ion/002_integration-angular.mdx Co-authored-by: Matt Walker <43502076+matteematt@users.noreply.github.com>
…ion/003_integration-react.mdx Co-authored-by: Matt Walker <43502076+matteematt@users.noreply.github.com>
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.
This dependency injection explanation is great. Just a nitpic, it doesn't necessarily confirm the snippet as a general solution. If it is then you can tweak it as suggested (and do that for the react page too)
docs/001_develop/03_client-capabilities/022_framework-integration/002_integration-angular.mdx
Outdated
Show resolved
Hide resolved
docs/001_develop/03_client-capabilities/022_framework-integration/002_integration-angular.mdx
Show resolved
Hide resolved
…ion/002_integration-angular.mdx Co-authored-by: Matt Walker <43502076+matteematt@users.noreply.github.com>
…ion/002_integration-angular.mdx Co-authored-by: Matt Walker <43502076+matteematt@users.noreply.github.com>
# Conflicts: # docs/001_develop/03_client-capabilities/022_framework-integration/002_integration-angular.mdx # docs/001_develop/03_client-capabilities/022_framework-integration/003_integration-react.mdx # docs/001_develop/03_client-capabilities/022_framework-integration/index.mdx
Draft for framework integration. Will add styling sections when genta is finished