Conversation
juliaam
left a comment
juliaam
left a comment
There was a problem hiding this comment.
o layout não está como esperado, a estrela está mais escura do que deveria e o tamanho do header da sidebar está pequeno, do maincomposer também
|
|
DominMFD
left a comment
DominMFD
left a comment
There was a problem hiding this comment.
Os headers ainda estão com a altura proporcional diferente
como ta:
figma:
Aparentemente o header da social media é um pouco menor do que como esta, só ajustar e ta bala
JpBurgarelli
left a comment
JpBurgarelli
left a comment
There was a problem hiding this comment.
Em vermelho está o que você usou/como está e em verde o que deveria estar, com base no layout do Figma:
Obs.: Dê uma conferida no tamanho dos ícones também.
juliaam
requested review from
DominMFD,
Frompaje,
JpBurgarelli,
Luiginicoletti,
PiluVitu and
juliaam
src/pages/home/components/Sidebar/components/SocialAccordion/SocialAccordion.module.scss
Outdated
Show resolved
Hide resolved
src/pages/home/components/Sidebar/components/SocialAccordion/SocialAccordion.module.scss
Outdated
Show resolved
Hide resolved
|
Cade a thread? |
wendesongomes
force-pushed
the
refactorStyleSidebar#591
branch
from
35bf777 to
8169b4f
Compare
| const socialMediaName = socialMedias | ||
| .get(props.socialMediaId) | ||
| ?.name.toLowerCase(); | ||
| const iconPath = iconMap[socialMediaName as keyof typeof iconMap]; |
There was a problem hiding this comment.
as keyof typeof vou morrer. Pq precisamos disso?
juliaam
left a comment
juliaam
left a comment
There was a problem hiding this comment.
figma:
como está na sua branch:
consegue ajustar o hover?
| "token": "DISCORD_EXAMPLE_TOKEN_1", | ||
| "valid": true, | ||
| "avatar": "https://example.com/image1.jpg" | ||
| "avatar": "https://cdn0.iconfinder.com/data/icons/free-social-media-set/24/discord-256.png" |
There was a problem hiding this comment.
é pra usar o ícone de um arquivo local do projeto, vamos conversar sobre
There was a problem hiding this comment.
e o ícone também não tá condizente com o do figma
| .username { | ||
| margin: 0; | ||
| } | ||
|
|
There was a problem hiding this comment.
muito bom! deixou alinhadinho ao layout!
There was a problem hiding this comment.
isso não deve ficar aqui, todos os icons estão em src/components/Icons/icons, e já tem o do discord
| const renderAccordionContent = (): ReactNode => ( | ||
| <ul role="listitem">{renderAccordionMap()}</ul> | ||
| <ul className={scss.containerAccordion} role="listitem"> | ||
| {renderAccordionMap()} |
There was a problem hiding this comment.
não é mais assim no code-style atual, tem que colocar no .components
JpBurgarelli
left a comment
JpBurgarelli
left a comment
There was a problem hiding this comment.
Pelo que me recordo, esse botao da estrela estava pegando, mas, agora, nao funcionou.
O nome 'e o icone da rede social deveria ser roxo, nao? Olha o layout:
juliaam
removed
AGUARDANDO ALE
Changes Requested
6 participants
Closes #591
Feature
Este PR corrige o problema onde o estilo da sidebar havia sido pedido, retornando-a ao seu design padrão. Após algumas modificações, o estilo original foi perdido, e esta atualização restaura o layout e a aparência esperados para garantir consistência na interface do usuário.
Visual evidences 🖼️