Feature/layout da home e projetos #35
Conversation
Co-authored-by: Frederico <frederico.smozzato@gmail.com> Co-authored-by: Moisés <moisesssprg@gmail.com>
Co-authored-by: Frederico <frederico.smozzato@gmail.com> Co-authored-by: Moisés <moisesssprg@gmail.com>
Co-authored-by: Lucas Vasques <Luckvc@users.noreply.github.com>
Co-authored-bty: Lucas Vasques <lucasvc1996@gmail.com>
Co-authored-by: Lucas Vasques <lucasvc1996@gmail.com>
Co-authored-by: Lucas Vasques <Luckvc@users.noreply.github.com>
Co-authored-by: Lucas Vasques <Luckvc@users.noreply.github.com>
Co-authored-by: Lucas Vasques <lucasvc1996@gmail.com>
fredericomozzato
changed the title
There was a problem hiding this comment.
App tá ficando bem bonita, o que é bem bacana. Um ponto de atenção importante gente é não misturar refatorações com a questão do layout. Mexer no layout já é algo enorme por si só (vide o número de mudanças) e adicionar refatorações de código que não tem a ver com layout da app pode fazer com que esse código passe batido, além de dar ao PR responsabilidades que ele não deve ter (visto que ele deve ser restringido a determinada funcionalidade).
No futuro, caso percebam que é necessário fazer alguma alteração em código antes de mexer no layout, abram outra Issue e resolvam ela em um PR separado, para que sigamos as boas práticas direitinho, tudo bem?
Além disso, para PRs futuros, também seria bacana dividir a refatoração do CSS por área, visto o tanto de alterações que foram necessárias. Um para a Home, outro para os Projetos, e assim por diante :)
No geral, bom trabalho!
fredericomozzato
requested review from
oLucasAguilar,
mateuscavedini,
adoniranfranceh and
paulohenrique-gh
Alcançado com este PR
Resolve as issues #30, #31 e #43.
Página inicial
Foi criada uma página inicial com CSS para visitantes não logados que exibe informações sobre a aplicação e um botão para login ou criação de conta. Além disso foi adicionado um favicon com a logo do projeto que é exibido nas abas:
Ambos formulários de login e cadastro foram estilizados com Bootstrap:
Formulário de login:
Formulário de cadastro:
Redirecionamento para projetos
Ao fazer login, um usuário é redirecionado para a página de seus projetos. Ele tem acesso somente aos projetos que é dono. A página foi estilizada com Bootstrap e os projetos são exibidos em formato de cards:
Visão detalhada de projeto
A página de cada projeto individual contém uma aba de navegação para o usuário acessar os itens de projeto:
Cada opção leva para uma tela detalhada. A aba conta com um feedback para informar qual seção do projeto o usuário está visualizando.
Padronização de telas para itens de projeto
Os itens de projeto atualmente implementados são Tarefas, Reuniões e Documentos. Cada página foi estilizada com Bootstrap usando um layout em formato de tabela.
Tela de tarefas:
Tela de Reuniões:
Tela de documentos:
Flash messages
Foi adicionado estilo nas flash messages para dar mais destaque em mensagens de sucesso e erro:
Flash de sucesso:
Flash de erro:
Mudança da rota de index de projetos
A rota my_projects foi removida provisóriamente pois todos os projetos do usuário ficarão dentro da index de projetos. Sendo assim, a rota se tornou redundante e foi removida.
Débitos
Não foi feita a estilização da página de pesquisa de usuários. Além disso não foi implementada uma tela para que o usuário tenha acesso a projetos que ele é colaborador, garantindo acesso somente aos que o usuário é dono.