-
-
Notifications
You must be signed in to change notification settings - Fork 20
Feat/create header component #192
Feat/create header component #192
Conversation
@franklingg e @NicolasLopes7 Gente tinha esquecido de fazer o design do mobile, aí fiz aqui: Tanto o footer como o header tem conteúdos que são repetidos retirados, já que eles não eram estritamente necessários |
src/components/Header/styles.jsx
Outdated
line-height: 3rem; | ||
|
||
&:hover { | ||
transform: scale(1.05); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acredito que seria muito legal aplicar esse efeito de transform nos ícones abaixo:
para o hover dos links é legal ser a cor #FC8E51. O que achas @franklingg?
src/components/Header/styles.jsx
Outdated
padding: 3.7rem 4.4rem; | ||
background-color: rgba(252, 252, 252, 0.6); | ||
|
||
@media (max-width: 428px){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@franklingg por favor, faz um |
# Conflicts: # package.json
# Conflicts: # .babelrc # src/styles/globals.css
@@ -25,3 +26,6 @@ html { | |||
font-size: 62.5%; | |||
} | |||
|
|||
body { | |||
font-size: 1.6rem; | |||
background: url("../assets/img/background.png") repeat; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ooie @franklingg primeiramente parabénssss! Tá incrível a tela inicial. Amei muito!!!!
Vou pontuar dois errinhos básicos de compilação que tive e que são rápidos de consertar.
Na linha 32 temos a falta da chave de fechamento do body
peço que adicione por favorzinho 🚀
@@ -3,10 +3,14 @@ import LanguageDetector from 'i18next-browser-languagedetector'; | |||
import { initReactI18next } from 'react-i18next'; | |||
|
|||
import translationsPTBR from './locales/pt-br.json'; | |||
import translationsENUS from './locales/en-us.json'; | |||
import translationsESES from './locales/es-es.json'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4f8c031
to
0b6838b
Compare
0b6838b
to
bd25539
Compare
Issue
closes #182
Qual o objetivo dessa Pull Request?
Criar o componente Header. Além disso, permite alterar as línguas para inglês e espanhol.
Como pode ser manualmente testado?
É preciso rodar
yarn ou npm install
por ter novas dependências (react-select
ebabel
), antes de rodaryarn dev