Destacar página atual no cabeçalho

[guites]

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

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

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

[OliverLidi]

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

[guites]

Claro!! Qualquer dúvida é só falar. Quando eu estiver no pc te atribuo a issue 🤠 Em sáb., 5 de out. de 2024 11:04, OliverLidi ***@***.***> escreveu:

…

Olá bom dia, gostaria de pegar essa tarefa. tudo bem? — Reply to this email directly, view it on GitHub <#80 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ARFGRE4IKMTSPAINX7FLMPDZZ7WXXAVCNFSM6AAAAABPIUIMA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGA3DSNJQHA> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[OliverLidi]

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

[guites]

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]

Oii Guites, fiz as alterações.

[guites]

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]

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

[guites]

Eu não tô no Pc agora, mas repara que no arquivo tem umas marcações no estilo `<<<<<<< main` e também `======`. Isso indica que deu algum erro na hora de resolver os conflitos de merge. Se você tentar rodar o projeto vai reparar que a página vai dar erro. Consegue fazer esse teste? Daí vai ficar mais claro pra ti o que precisa ser arrumado. Em sáb., 19 de out. de 2024 12:46, OliverLidi ***@***.***> escreveu:

…

Oi Guites, preciso fazer algo mais? fiz o pull request certo? — Reply to this email directly, view it on GitHub <#80 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ARFGRE54QMFN5WPLFX6SUW3Z4J5E3AVCNFSM6AAAAABPIUIMA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRTHE4TCNJZGA> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[sylvester-96]

Essa issue ja foi resolvida?

[guites]

@sylvester-96 ainda não, faltaram uns ajustes no PR da @OliverLidi . Vamos aguardar pra ver se ela ainda vai mexer :)

[OliverLidi]

Oppa, gente! Sábado eu finalizo . Tudo bem?

[guites]

Oppa, gente! Sábado eu finalizo . Tudo bem?

Tranquilo!!! Como ocorreram várias mudanças no arquivo que você está mexendo, talvez seja mais fácil sincronizar o teu fork com as novas alterações e abrir o PR novamente!

[OliverLidi]

Oie, fiz as modificações. Será que agora foi certo? rs