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

[SW-2944] Sidebar component #28

Merged
merged 4 commits into from
Aug 6, 2023
Merged

Conversation

VHvictor1
Copy link
Contributor

@VHvictor1 VHvictor1 commented Aug 1, 2023

Descrição

Tentativa de criação do componente sidebar com a mesma estrutura do flowbite.

Observações

Ainda está faltando o Dropdown, a variante para o mobile e os testes que não foram possíveis de fazer essa semana.

Prints

image
image

Checklist

  • Fiz o link com a task do clickup.
  • Fiz minha própria revisão do código.
  • Realizei os testes que compravam que a funcionalidade está funcionando corretamente.

create the sidebar component
@PedroAugustoRamalhoDuarte
Copy link
Member

@VHvictor1 Depois me manda uma foto de como ficou, pelo código parece que estamos bem alinhados mesmo no formato de uso

@PedroAugustoRamalhoDuarte
Copy link
Member

@VHvictor1 Parece tudo certo, gostaria de fazer um pedido, tu poderia fazer um exemplo mais real copiando o do figma, por exemplo. Tenho uma pergunta também ta funcionando o Dropdown e o modo mobile nessa PR? Podemos remanejar também para outras task caso tenha passado muito o tempo justo. E vou deixar o @LordeFelipe revisar com mais calma

@VHvictor1
Copy link
Contributor Author

VHvictor1 commented Aug 3, 2023

@LordeFelipe Acho que faz sentido criar mais duas tasks para esse dropdown e para a outra variante da side bar pois já estou a bastante tempo tentando resolver esse componente e coisas bastante simples estão dando errado.

create the side bar component with for desktop
@VHvictor1 VHvictor1 changed the title [Draft] Sidebar component [SW-2944] Sidebar component Aug 3, 2023
@VHvictor1 VHvictor1 requested review from LordeFelipe and removed request for PedroAugustoRamalhoDuarte August 3, 2023 14:40
Copy link

@LordeFelipe LordeFelipe left a comment

Choose a reason for hiding this comment

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

@VHvictor1 Ficou muito bom, acho q as coisas q vc falou podem ficar de task pra semana que vem mesmo. Só queria saber se vc consegue colocar pra ser possivel customizar as cores da sidebar.

@PedroAugustoRamalhoDuarte
Copy link
Member

PedroAugustoRamalhoDuarte commented Aug 6, 2023

@LordeFelipe Vou deixar as cores para essa semana, valeu pelo review

@PedroAugustoRamalhoDuarte PedroAugustoRamalhoDuarte merged commit 9ebccfc into main Aug 6, 2023
1 check passed
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.

3 participants