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 do Frontend #602

Merged
merged 1 commit into from
Jul 29, 2022

Conversation

filipedeschamps
Copy link
Owner

@filipedeschamps filipedeschamps commented Jul 29, 2022

Este PR reúne a atualização de todas as dependências (módulos) do Frontend.

Dependências não "totalmente" atualizadas do frontend

  • recharts fiz o upgrade de 2.1.9 para 2.1.12 pois a 2.1.13 quebra por conta disso

Efeitos colaterais

Não há testes automatizados no frontend então foi tudo teste manual mesmo e a única coisa que quebrou aparentemente na questão visual foi a atualização do @primer/react de 35.2.1 para 35.5.0 por conta desse release que agora faz o TextInput (e outros Inputs pel queo testei) não preencher por padrão a largura do container, você precisa controlar isso pela propriedade block.

Em paralelo, o componente Button deles também tomou esse hit, mas ele não aceita a propriedade block ainda. Tanto que na documentação eles estão recomendando estilizar com o sx.

O Textarea do Bytemd também foi afetado.

@vercel
Copy link

vercel bot commented Jul 29, 2022

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

Name Status Preview Updated
tabnews ✅ Ready (Inspect) Visit Preview Jul 29, 2022 at 3:11PM (UTC)

filipedeschamps added a commit that referenced this pull request Jul 29, 2022
* `recharts` was upgraded only to `2.1.12`, more info at PR #602
@filipedeschamps filipedeschamps force-pushed the dependencies-frontend branch from 37b03be to 939ad7d Compare July 29, 2022 13:50
@aprendendofelipe
Copy link
Collaborator

O Textarea do Bytemd também foi afetado.

Também tem que arrumar o css :focus-within, ou seja, ajustar o border e retirar box-shadow para ficar igual ao novo padrão do @primer/react.

Lembra no PR #462 do motivo de deixar esses comentários?

{/* 36px is the size of the TabCoin column */}
<Box sx={{ width: 'calc(100% - 36px)' }}>

{/* 36px is the size of the TabCoin column */}
<Box sx={{ width: 'calc(100% - 36px)' }}>

Agora que retiraram o box-shadow, talvez não ocorra mais aquele problema ao editar o conteúdo root e, se realmente for o caso, não tem mais necessidade do calc(100% - 36px)

@filipedeschamps
Copy link
Owner Author

filipedeschamps commented Jul 29, 2022

Também tem que arrumar o css :focus-within, ou seja, ajustar o border e retirar box-shadow para ficar igual ao novo padrão do @primer/react.

Nossa é verdade! Não tinha percebido essa diferença visual. Que pena, e achava tão bonita a sombra ao redor 😂

Eles fazem parte da borda com uma sombra com essa regra:

.fwAUCe:focus-within {
    border-color: #0969da;
    outline: none;
    box-shadow: inset 0 0 0 1px #0969da;
}

image

Fica bugado no Bytemd, investigando 🤝

Agora que retiraram o box-shadow, talvez não ocorra mais aquele problema ao editar o conteúdo root e, se realmente for o caso, não tem mais necessidade do calc(100% - 36px)

Eu acho que vai dar pra retirar heim!!! 😍

@filipedeschamps
Copy link
Owner Author

E o problema de usar border-width: 2px; para evitar aquele glitch é esse:

Screen.Recording.2022-07-29.at.7.21.06.AM.mov

@filipedeschamps
Copy link
Owner Author

filipedeschamps commented Jul 29, 2022

Tem um "hackzinho de layout" ali, vamos ver se alguém nota:

Screen.Recording.2022-07-29.at.8.06.06.AM.mov

[edit]

A borda também ficou levemente mais escura

* `recharts` was upgraded only to `2.1.12`, more info at PR #602
@filipedeschamps filipedeschamps force-pushed the dependencies-frontend branch from 939ad7d to 1c0f3b6 Compare July 29, 2022 15:09
@filipedeschamps filipedeschamps merged commit 281515b into main Jul 29, 2022
@filipedeschamps filipedeschamps deleted the dependencies-frontend branch July 29, 2022 16:34
@filipedeschamps
Copy link
Owner Author

Merged!!! Let's goooo!!!

Em paralelo, sobre esse hack:

{/* 36px is the size of the TabCoin column */}
<Box sx={{ width: 'calc(100% - 36px)' }}>

Sugiro fazermos em outro PR 🤝 provavelmente podemos juntar tudo na tarefa Compactar espaçamentos em Mobile (principalmente na árvore de respostas).

@aprendendofelipe
Copy link
Collaborator

Tem um "hackzinho de layout" ali, vamos ver se alguém nota:

Achei simpática essa barra faltando um pixel de cada lado e a borda branca no toolbar causada pelo padding.


Sugiro fazermos em outro PR 🤝 provavelmente podemos juntar tudo na tarefa Compactar espaçamentos em Mobile (principalmente na árvore de respostas).

Isso, esse PR vai exigir mudanças nesse componente.

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