Skip to content

Conversation

@mariana-caetano
Copy link
Collaborator

@mariana-caetano mariana-caetano commented Oct 30, 2025

Description

This PR fixes sidebar styling issues affecting navigation on smaller screens.

Now, the sidebar stays open regardless of screen size, and users can close it whenever they want. The solution aligns with the issue raised by the design team: vtexdocs/helpcenter#365

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which has been updated accordingly.

@mariana-caetano mariana-caetano self-assigned this Oct 30, 2025
@mariana-caetano mariana-caetano added the release-auto Automatic version bump label Oct 30, 2025
@mariana-caetano mariana-caetano marked this pull request as ready for review November 12, 2025 14:26
@PedroAntunesCosta
Copy link
Collaborator

Testei essa PR atualizando o package.json do projeto helpcenter para apontar para a branch fix/sidebar:

"@vtexdocs/components": "https://github.com/vtexdocs/components.git#fix/sidebar"

Depois executei yarn e yarn dev.

Problema Encontrado

A aplicação falha ao renderizar com o seguinte erro:

Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined.

Parece que o projeto helpcenter importa CopyIcon e SearchIcon de @vtexdocs/components, mas esses ícones não estão sendo exportados no index principal do pacote.

Sugestão

Adicionar os exports dos ícones que estão faltando em src/index.ts:

export { default as CopyIcon } from './components/icons/copy-icon'
export { default as SearchIcon } from './components/icons/search-icon'

Assim que corrigido, posso re-testar as mudanças de comportamento da sidebar.

@carolinamenezes carolinamenezes deleted the fix/sidebar branch December 10, 2025 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-auto Automatic version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants