Skip to content

Conversation

GabrielPassos25
Copy link
Contributor

Baseado na issue: RedeMapas/mapas#1098

  • Novo design do menu de usuário (Mobile):
image
  • Novo design do menu de usuário (Desktop):
image
  • Painel de controle (Admin):
image

@GabrielPassos25 GabrielPassos25 self-assigned this Oct 7, 2025
Copy link
Member

@lpirola lpirola left a comment

Choose a reason for hiding this comment

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

para uma alteração relativamente pequena, foi preciso alterar muitos arquivos.

temos adotado uma estratégia de evitar sobrescrever os componentes existenets.

damos preferência para o uso do hook e nesse caso acho que pode ajudar muito.

consegue tentar?

@GabrielPassos25
Copy link
Contributor Author

GabrielPassos25 commented Oct 9, 2025

para uma alteração relativamente pequena, foi preciso alterar muitos arquivos.

temos adotado uma estratégia de evitar sobrescrever os componentes existenets.

damos preferência para o uso do hook e nesse caso acho que pode ajudar muito.

consegue tentar?

@lpirola qual seria o hook? não sabia que teve mudança nessa forma dos arquivos

@lpirola
Copy link
Member

lpirola commented Oct 13, 2025

https://vscode.dev/github/redemapas/theme-funarte/blob/feat/circuitos/Theme.php#L74

Esse link leva direto para linha do arquivo Theme.php no repositório que alterar a lista de links por hook.

Não tenho certeza que todas alterações podem ser feitas por hook mas a preocupação é em alterarmos muitas coisas e elas quebrarem quando houver atualização no arquivo fonte, no caso, o panel-nav.

@GabrielPassos25
Copy link
Contributor Author

https://vscode.dev/github/redemapas/theme-funarte/blob/feat/circuitos/Theme.php#L74

Esse link leva direto para linha do arquivo Theme.php no repositório que alterar a lista de links por hook.

Não tenho certeza que todas alterações podem ser feitas por hook mas a preocupação é em alterarmos muitas coisas e elas quebrarem quando houver atualização no arquivo fonte, no caso, o panel-nav.

@lpirola beleza, vou tentar e qualquer coisa te aviso!

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