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(footer): implement footer component #173

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

olirock
Copy link

@olirock olirock commented Sep 16, 2024

Descrição

Criação do componente Footer com layouts para Web e Mobile(responsivo) de acordo com template definido no figma

Desktop
Captura de Tela 2024-09-16 às 15 28 43

Mobile
Captura de Tela 2024-09-16 às 15 30 34


Tipo da alteração

feat: Uma nova funcionalidade


Checklist

  • Revi meu próprio código antes de enviar para revisão
  • Testei minha correção ou recurso extensivamente antes de enviar para revisão
  • Formatei meu código usando o guia de estilo de interface (Prettier + ESLint)
  • Meu código está bem documentado com apenas informações relevantes
  • Minhas alterações não geram novos avisos
  • Meus commits são bem organizados, descritivos e fáceis de reverter
  • Adicionei testes que provam que minha correção é eficaz ou que meu recurso funciona
  • Meu código está seguindo os padrões de acessibilidade

Outras observações


Copy link

github-actions bot commented Sep 16, 2024

Deploy preview for portal-ascepa ready!

✅ Preview
https://portal-ascepa-h5zyblxec-andrew-rosarios-projects.vercel.app

Built with commit 9c936cc.
This pull request is being automatically deployed with vercel-action

@Werisu Werisu self-requested a review September 16, 2024 22:50
Copy link

sonarcloud bot commented Sep 18, 2024

Copy link
Contributor

@FelipeDuarteLuna FelipeDuarteLuna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa Tarde Guerreiro, Firme ?

Sua implementação funcionou legal, mas precisa realizar 2 ajustes.
I. Utilizar o Tokens de margens e cor;
II. Ajuste o Media Queries, com px não fica renderizando conforme o tamanho.
image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa Tarde Guerreiro, Firme ?

Sua implementação funcionou legal, mas precisa realizar 2 ajustes.
I. Utilizar o Tokens de margens e cor;
II. Ajuste o Media Queries, com px não fica renderizando conforme o tamanho.
image


footer {
text-align: center;
padding: 20px;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remover padding:20px

@@ -0,0 +1,10 @@
<footer id="footer">
Copy link
Member

@Werisu Werisu Sep 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

adicionar a class p-20px e mt-20px

text-align: center;
padding: 20px;
background-color: #f8f8f8;
margin-top: 20px;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remover margin-top: 20px

footer {
text-align: center;
padding: 20px;
background-color: #f8f8f8;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acredito que não exista essa cor no projeto do figma

projeto figma

utilizar o token $color-white

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants