Skip to content

Conversation

plagoa
Copy link
Contributor

@plagoa plagoa commented Aug 18, 2025

We originally left the playground controls section out of the inner HvProvider but in reality we need it there as some of the controls (the HvColorPicker) depend on it to get the correct colors.

@plagoa plagoa requested a review from a team as a code owner August 18, 2025 16:47
@plagoa plagoa requested review from zettca and francisco-guilherme and removed request for a team August 18, 2025 16:47
@github-actions github-actions bot temporarily deployed to uikit-docs/pr-4871 August 18, 2025 16:51 Destroyed
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR moves the DocsProvider wrapper from only covering the preview area to encompass both the preview and controls sections. This change ensures that controls like HvColorPicker can access the correct theme colors through the provider context.

  • Restructured the layout to wrap both preview and controls sections with DocsProvider
  • Added background styling to the controls area for visual consistency
  • Maintained the existing grid layout structure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@francisco-guilherme francisco-guilherme merged commit a50a2c7 into master Sep 4, 2025
14 checks passed
@francisco-guilherme francisco-guilherme deleted the docs/provider_controls branch September 4, 2025 08:59
francisco-guilherme added a commit that referenced this pull request Sep 11, 2025
* docs: add HvProvider to the Controls section

* Update apps/docs/src/components/code/Playground.tsx

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: francisco-guilherme <[email protected]>
Co-authored-by: Copilot <[email protected]>
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