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

feat(header): set header link to active #675

Merged
merged 2 commits into from
Aug 24, 2022

Conversation

gpaiva00
Copy link
Contributor

@gpaiva00 gpaiva00 commented Aug 23, 2022

objetivo

Indicar em qual página estamos

o que mudou?

  • estilização padrão para quando o header link estiver ativo
  • verificação em cada link para saber se ele deve ser ativo ou não
  • Trocar TabNews por Relevantes
  • esconder Status quando estiver no Mobile

evidencias

@gpaiva00 gpaiva00 added the front Envolve modificações no frontend label Aug 23, 2022
@gpaiva00 gpaiva00 self-assigned this Aug 23, 2022
@vercel
Copy link

vercel bot commented Aug 23, 2022

Someone is attempting to deploy this pull request to the TabNews Team on Vercel.

To accomplish this, the commit author's email address needs to be associated with a GitHub account.

Learn more about how to change the commit author information.

@vercel
Copy link

vercel bot commented Aug 23, 2022

@gpaiva00 is attempting to deploy a commit to the TabNews Team on Vercel.

To accomplish this, @gpaiva00 needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@filipedeschamps
Copy link
Owner

Ótima primeira contribuição @gpaiva00 🎉

Três sugestões:

  1. Fazer squash nos seus commits e fazer um push f para cá.
  2. E mudar algo que eu já estava querendo fazer antes, que era alterar o texto do iten TabNews para Relevantes. Apesar que isso tira um pouco do brand do TabNews, vai fazer muito mais sentido para quem está usando o site.
  3. Hoje em Mobile, está sendo escondido escondendo o TabNews, e agora sugiro esconder o Status porque devemos passar esse link para um Footer que ainda não existe, mas devemos criar dentro dessa Milestone para conseguir colocar outros links importantes 👍 A responsividade está sendo controlada aqui

@gpaiva00
Copy link
Contributor Author

Ótima primeira contribuição @gpaiva00 🎉

Três sugestões:

  1. Fazer squash nos seus commits e fazer um push f para cá.
  2. E mudar algo que eu já estava querendo fazer antes, que era alterar o texto do iten TabNews para Relevantes. Apesar que isso tira um pouco do brand do TabNews, vai fazer muito mais sentido para quem está usando o site.
  3. Hoje em Mobile, está sendo escondido escondendo o TabNews, e agora sugiro esconder o Status porque devemos passar esse link para um Footer que ainda não existe, mas devemos criar dentro dessa Milestone para conseguir colocar outros links importantes 👍 A responsividade está sendo controlada aqui

Perfeito, @filipedeschamps! Vou implementar 👍🏽

@gpaiva00 gpaiva00 force-pushed the feat-header-link-to-active branch from 1bfaf91 to d2417f6 Compare August 24, 2022 19:28
@filipedeschamps
Copy link
Owner

Show de bola @gpaiva00 vou fazer o squash em uma branch a parte para forçar o deploy na Vercel, e depois fazer o merge na main 🤝

@filipedeschamps filipedeschamps changed the base branch from main to preview August 24, 2022 21:53
@filipedeschamps filipedeschamps merged commit b9743b1 into preview Aug 24, 2022
@filipedeschamps filipedeschamps deleted the feat-header-link-to-active branch August 24, 2022 21:54
filipedeschamps added a commit that referenced this pull request Aug 25, 2022
feat(header): set header link to active (#675)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front Envolve modificações no frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants