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

Draft - Framework integration #1980

Merged
merged 16 commits into from
Nov 29, 2024

Conversation

patrickoneill-genesis
Copy link
Contributor

Draft for framework integration. Will add styling sections when genta is finished

Copy link
Contributor

@matteematt matteematt left a 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


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`.
Copy link
Contributor

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

Copy link
Contributor

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>
Copy link
Contributor

@matteematt matteematt left a 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)

…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
@patrickoneill-genesis patrickoneill-genesis merged commit c5416a9 into new-doc-structure Nov 29, 2024
@patrickoneill-genesis patrickoneill-genesis deleted the pon/framework-integration branch November 29, 2024 16:58
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