-
Notifications
You must be signed in to change notification settings - Fork 0
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
Inicia implementação do breadcrumb. #57
Conversation
Co-authored-by: Thalyta Lima <[email protected]>
Co-authored-by: Thalyta Lima <[email protected]>
Co-authored-by: Thalyta Lima <[email protected]>
Co-authored-by: Thalyta Lima <[email protected]>
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.
Deixei pre-aprovado mas deixei alguns comentarios que seria legal dar uma olhada!
spec/system/common_area/manager_registers_new_common_area_spec.rb
Outdated
Show resolved
Hide resolved
Co-authored-by: Thalyta Lima <[email protected]>
Co-authored-by: Lucas Lima <[email protected]>
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.
Deixei alguns comentários com sugestões para melhoria do código, mas são detalhes, de resto não vi nada de errado na implementação, por isso tem minha aprovação.
Este PR inclui a visualização do breadcrumb nas views da aplicação, para auxiliar o usuário a se localizar e navegar entre as páginas.
Para implementação dos breadcrumbs, utilizamos a gem Breadcrumbs on Rails, que configura os dados do breadcrumb dentro dos controllers da aplicação. O caminho é exibido nas views, exceto naquelas que compõem do home_controller, através de uma partial.
Foram feitos alguns testes de navegação para a nova funcionalidade.
Tela de cadastro de condomínio, com breadcrumb:
Navegação entre as páginas da aplicação usando os breadcrumbs:
Resolve Issue #52