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

chore(rna): theming docs #3060

Merged
merged 7 commits into from
Nov 22, 2022
Merged

chore(rna): theming docs #3060

merged 7 commits into from
Nov 22, 2022

Conversation

dbanksdesign
Copy link
Contributor

Description of changes

Issue #, if available

Description of how you validated changes

Checklist

  • PR description included
  • yarn test passes
  • Tests are updated
  • No side effects or sideEffects field updated
  • Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dbanksdesign dbanksdesign requested a review from a team as a code owner November 22, 2022 05:54
@changeset-bot
Copy link

changeset-bot bot commented Nov 22, 2022

⚠️ No Changeset found

Latest commit: 96ca110

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@@ -184,19 +184,16 @@ const SecondaryNav = (props) => {
))}
</ExpanderItem>

{/* React Native does not yet support theming */}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Woot!

@@ -9,7 +9,3 @@ The Authenticator has an option to hide the sign up page including the `Create A
<Fragment>
{({ platform }) => import(`./hidesignup.default.${platform}.mdx`)}
</Fragment>

<Example>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed this since it was rendering the React web authenticator


You can update the style of the Authenticator by wrapping it with a [ThemeProvider](/react-native/theming). Then create a [theme object](/react-native/theming#theme-structure), with all your font customizations.

```jsx file=../../../../../../../examples/react-native/src/features/Authenticator/Theming/Example.tsx
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I assume this is a good idea so we can test the example even if we aren't rendering it in the docs

Copy link
Member

@calebpollman calebpollman left a comment

Choose a reason for hiding this comment

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

LGTM, left a couple suggestions in teh examples

dbanksdesign and others added 2 commits November 21, 2022 23:27
…le.tsx

Co-authored-by: Caleb Pollman <cpollman@amazon.com>
…ode.tsx

Co-authored-by: Caleb Pollman <cpollman@amazon.com>
@calebpollman calebpollman merged commit 3018b6c into rna/docs-setup Nov 22, 2022
@calebpollman calebpollman deleted the rna-theming-docs branch November 22, 2022 07:31
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.

3 participants