Removi o header do AOSC Fest #65
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Olá @sukuasoft, valeu pela PR.
Além do meu comentário específico, eu gostaria de informar que o menu já na está a funcionar conforme devia na plataforma mobile.
Você podia dar uma olhada ou um testesinho?
| @@ -47,7 +47,7 @@ function Header({ ...rest }) { | |||
|
|
|||
| return ( | |||
| <header | |||
| className="dark:bg-bgd w-full flex-col items-center justify-center px-6 hidden lg:flex fixed top-10 bg-white z-50" | |||
| className="dark:bg-bgd w-full flex-col items-center justify-center px-6 hidden lg:flex fixed bg-white z-50" | |||
There was a problem hiding this comment.
Por quê removeu o top-10?
There was a problem hiding this comment.
Este top-10, adiciona um padding no topo do header. Servia para dar uma distancia entre header e a informação do AOSC Fest que ficava por cima.
Mas como removi header do AOSC Fest não necessidade do top-10 ainda estar, porque senão ficará assim o header:
There was a problem hiding this comment.
Sim, deu uma olhada no menu. Está do seguinte jeito:
There was a problem hiding this comment.
@antonio-pedro99 Por engano adicionei mais um commit, aqui. Era sobre a questão das métricas. Tem como fazer um rollback.
|
@sukuasoft o building do vercel falhou, podes dar uma olhada? |
Neste PR removi o header do AOSC Fest, porque o evento já passou, assim como solicitado na issue 64.
Alterações: