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

Atualiza todas as dependências possíveis #1733

Merged
merged 5 commits into from
Jun 25, 2024

Conversation

aprendendofelipe
Copy link
Collaborator

Foram atualizadas todas as dependências possíveis.

Não atualizadas

  • eslint: só até v8.57.0 por compatibilidade com eslint-config-next;
  • eslint-plugin-vitest: só até v0.4.1 por compatibilidade com .eslintrc (alguns plugins ainda não são compatíveis com eslint.config.js);
  • kill-port: por causa do bug;
  • next-connect: por exigir mudanças profundas no código e não estar claro se é compatível com o Next 14;
  • styled-components: por compatibilidade com @primer/react;
  • uuid: por compatibilidade com vis-network;

Atualizadas com adequações

  • @react-email/components: precisou de atualizações dos snapshots por causa da nova meta tag x-apple-disable-message-reformatting;

@primer/react

Havia um bug no TextInput.Action que nos impedia de utilizá-lo no PasswordInput, mas agora o bug foi corrigido e não precisamos mais contornar isso, então reverti a adequação provisória.

Houve uma mudança estética no Tooltip do GitHub, que passou a ter um formato um pouco mais arredondado e não possui mais a seta.

Antes Depois
Tooltip antes Tooltip depois

Tipo de mudança

  • Atualização de dependências

Checklist:

  • As modificações não geram novos logs de erro ou aviso (warning).
  • Os testes antigos estão passando localmente.

Copy link

vercel bot commented Jun 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 25, 2024 4:31pm

Copy link
Collaborator

@Rafatcb Rafatcb left a comment

Choose a reason for hiding this comment

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

Curiosamente, eu estava vendo as dependências do projeto ontem de noite, então deixei alguns comentários sobre dependências que não são utilizadas.

Edit:

Percebi que na documentação, o Tooltip do Primer ficou bem mais próximo do gatilho do que o nosso, que é posicionado como se houvesse uma seta invisível. Tem algum estilo que precisamos modificar para corrigir isso?

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
@aprendendofelipe
Copy link
Collaborator Author

Curiosamente, eu estava vendo as dependências do projeto ontem de noite, então deixei alguns comentários sobre dependências que não são utilizadas.

Boa, já removi ou movi as dependências conforme seus comentários. 💪

Percebi que na documentação, o Tooltip do Primer ficou bem mais próximo do gatilho do que o nosso, que é posicionado como se houvesse uma seta invisível. Tem algum estilo que precisamos modificar para corrigir isso?

Acho que esqueceram de ajustar isso no Tooltip v1 quando removeram a seta. Estou sobrescrevendo as margens para 2px, que é o que pareceu ficar melhor em todos os locais que usamos o Toolltip.

Co-authored-by: Rafael Tavares Carvalho Barros <[email protected]>
@aprendendofelipe aprendendofelipe force-pushed the chore/update-dependencies branch from 2e34e3f to bd298ff Compare June 25, 2024 16:29
@aprendendofelipe aprendendofelipe merged commit 21f8070 into main Jun 25, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the chore/update-dependencies branch June 25, 2024 16:36
@Rafatcb
Copy link
Collaborator

Rafatcb commented Jun 25, 2024

Parece que algumas outras estilizações quebraram/mudaram:

Bloco de código com copiar (está cortado, não foi da print):

image

Citação e links:

image

@aprendendofelipe
Copy link
Collaborator Author

Parece que algumas outras estilizações quebraram/mudaram:

Vou investigar! Já fiz rollback 👍

@aprendendofelipe
Copy link
Collaborator Author

Até o Next.js 14.1.4 nada muda, mas a partir do 14.2.0 alguns estilos do ByteMD são alterados.

Imagino que algo deve ter mudado com relação ao style jsx

@Rafatcb
Copy link
Collaborator

Rafatcb commented Jun 25, 2024

Não esperava que isso fosse "culpa" do Next.js, mas procurando no Google, faz sentido. Veja o anúncio sobre a otimização do CSS na versão 14.2:

We don’t expect this change to negatively impact the majority of applications. However, if you see any unexpected styles when upgrading, please review your CSS import order as per the recommendations in our documentation.

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