-
Notifications
You must be signed in to change notification settings - Fork 407
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
9f8b8e4
to
6325fd4
Compare
There was a problem hiding this 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?
6325fd4
to
0b3d77a
Compare
Boa, já removi ou movi as dependências conforme seus comentários. 💪
Acho que esqueceram de ajustar isso no Tooltip v1 quando removeram a seta. Estou sobrescrevendo as margens para |
Co-authored-by: Rafael Tavares Carvalho Barros <[email protected]>
b368d7d
to
2e34e3f
Compare
2e34e3f
to
bd298ff
Compare
Vou investigar! Já fiz rollback 👍 |
Até o Next.js Imagino que algo deve ter mudado com relação ao |
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:
|
Foram atualizadas todas as dependências possíveis.
Não atualizadas
eslint
: só atév8.57.0
por compatibilidade comeslint-config-next
;eslint-plugin-vitest
: só atév0.4.1
por compatibilidade com.eslintrc
(alguns plugins ainda não são compatíveis comeslint.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 comvis-network
;Atualizadas com adequações
@react-email/components
: precisou de atualizações dos snapshots por causa da nova meta tagx-apple-disable-message-reformatting
;@primer/react
Havia um bug no
TextInput.Action
que nos impedia de utilizá-lo noPasswordInput
, 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.
Tipo de mudança
Checklist: