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

docs(FormElement): add docs & refactor component structure #4450

Merged
merged 3 commits into from
Nov 28, 2024

Conversation

zettca
Copy link
Member

@zettca zettca commented Nov 25, 2024

  • hoist HvFormElement to root, aligning with other components (this is needed for the current nextra doc generation)
  • keep HvLabel, HvWarningText, etc. as HvFormElement's sub-components, keeping alignment with other components
  • merge HvFormElement's contexts (and remove/refactor redundant .Provider .Consumer components)

Warning

please rebase, don't squash

@zettca zettca requested a review from a team as a code owner November 25, 2024 14:44
@zettca zettca requested review from plagoa and francisco-guilherme and removed request for a team November 25, 2024 14:44
@github-actions github-actions bot temporarily deployed to uikit/pr-4450 November 25, 2024 14:49 Destroyed
@github-actions github-actions bot temporarily deployed to uikit-docs/pr-4450 November 25, 2024 14:49 Destroyed
@github-actions github-actions bot temporarily deployed to uikit/pr-4450 November 25, 2024 18:00 Destroyed
@github-actions github-actions bot temporarily deployed to uikit-docs/pr-4450 November 25, 2024 18:00 Destroyed
@github-actions github-actions bot temporarily deployed to uikit/pr-4450 November 25, 2024 18:58 Destroyed
@github-actions github-actions bot temporarily deployed to uikit-docs/pr-4450 November 25, 2024 18:59 Destroyed
@github-actions github-actions bot temporarily deployed to uikit/pr-4450 November 25, 2024 19:28 Destroyed
@github-actions github-actions bot temporarily deployed to uikit-docs/pr-4450 November 25, 2024 19:28 Destroyed
@github-actions github-actions bot temporarily deployed to uikit/pr-4450 November 25, 2024 20:23 Destroyed
@zettca zettca changed the title refactor(FormElement): hoist component to root docs(FormElement): add docs & refactor component structure Nov 26, 2024
@@ -123,79 +79,10 @@ export default function Demo() {
}
```

### External error message
Copy link
Member Author

Choose a reason for hiding this comment

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

this is a awkward example to add for each form component. it should be added in a complete form example/template instead

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should add a callout on each form component documentation page to direct people to the FormElement page (not just on the related components section)

@francisco-guilherme francisco-guilherme merged commit 22ff660 into master Nov 28, 2024
12 checks passed
@francisco-guilherme francisco-guilherme deleted the docs/nextra branch November 28, 2024 10:18
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