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

Destacar página atual no cabeçalho #80

Open
guites opened this issue Oct 2, 2024 · 8 comments
Open

Destacar página atual no cabeçalho #80

guites opened this issue Oct 2, 2024 · 8 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers website Ajustes e melhorias no portal

Comments

@guites
Copy link
Collaborator

guites commented Oct 2, 2024

Vamos adicionar um destaque na página que o usuário está.

Captura de Tela 2024-10-02 às 19 55 57

Por exemplo se você estiver na /movies , a palavra Filmes deve ficar destacada no cabeçalho.

Captura de Tela 2024-10-02 às 19 57 50

A mesma coisa deve acontecer na versão pra celular. Por exemplo, na print acima, Em cartaz deveria estar destacado dentro do menu expandido.

O nosso tema tem essa funcionalidade por padrão, basta adiconar active no nav-link da página que você está.

Referência: https://www.gethalfmoon.com/docs/navbar/#full-example

Esse link explica melhor como descobrir em qual página você está dentro dos templates: https://stackoverflow.com/questions/8676455/flask-current-page-in-request-variable

@guites guites added enhancement New feature or request good first issue Good for newcomers website Ajustes e melhorias no portal labels Oct 2, 2024
@OliverLidi
Copy link

Olá bom dia, gostaria de pegar essa tarefa. tudo bem?

@guites
Copy link
Collaborator Author

guites commented Oct 5, 2024 via email

@OliverLidi
Copy link

Olá, adicionei os active no arquivo base.html , preciso adicionar algo mais?

@guites
Copy link
Collaborator Author

guites commented Oct 12, 2024

Olá, adicionei os active no arquivo base.html , preciso adicionar algo mais?

@OliverLidi tava dando uma olhada nas tuas alterações, tá quase lá.

Se a gente só colocar o active em todos os links, eles vão ficar todos destacados, sempre.

A gente precisa destacar só se a pessoa estiver na página. Dá pra fazer assim:

<a class="nav-link {% if request.path == url_for('screening.index') %} active {% end %}" href="{{url_for('screening.index')}}"
  >Em cartaz</a
>

A variável request.path salva a página atual do usuário, e daí podemos ir comparando com o apontamento dos links pra só colocar a classe active se a pessoa estiver naquela página.

@OliverLidi
Copy link

Oii Guites, fiz as alterações.

@guites
Copy link
Collaborator Author

guites commented Oct 18, 2024

opa! @OliverLidi acho que já pode abrir o pull request!

Tenta ver esse vídeo: https://www.youtube.com/watch?v=Du04jBWrv4A

Se tiver alguma dúvida, pode mandar aqui ou chamar no telegram que eu te boto no grupo dos devs: https://t.me/guites

@OliverLidi
Copy link

Oi Guites, preciso fazer algo mais? fiz o pull request certo?

@guites
Copy link
Collaborator Author

guites commented Oct 19, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers website Ajustes e melhorias no portal
Projects
None yet
Development

No branches or pull requests

2 participants