-
Notifications
You must be signed in to change notification settings - Fork 32
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
Refeito a tela de um partido #82
Conversation
Boa! Ficou excelente! |
src/components/Header/styles.scss
Outdated
@@ -15,6 +15,10 @@ header { | |||
align-items: center; | |||
background-color: #ffffff; | |||
filter: drop-shadow(0px 4px 10px rgba(0, 0, 0, 0.25)); | |||
|
|||
> div { |
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.
poderia remover aqui e por um padding top de 15px no header mesmo, vai dar o mesmo efeito e fica mais simples
background-color: #e5e5e5; | ||
color: #2f2f2f; |
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.
coloca esse color no svg dos icones no componente também (ou remove a prop color de lá)
<div className="partido"> | ||
<div> | ||
<h1> | ||
{partido.sigla} - {partido.nome} |
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.
no css desse h1 coloca um text-align: center pra no mobile ele centralizar tbm
src/pages/PartidoInfo/index.js
Outdated
<h1 className="title is-4"> | ||
{partido.nome} - {partido.sigla} | ||
<div className="divPartido"> | ||
<div className="infoPartido"> |
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.
nesses cards da pagina de partido, usa as colunas do bulma css pra quando for em tablet/celular ele quebrar linha e cada card ficar em 100% um embaixo do outro, qualquer coisa quando eu fizer o PR da minha parte da pra olhar lá como eu fiz, ou só dar uma ligada aqui: https://bulma.io/documentation/columns/responsiveness/
|
Fiz as alterações solicitadas. |
massa |
#76
Descrição do bug/feature:
Tela de um perfil de partido refeito.
Solução
Foi alterado o css da página.
Coisas a fazer
#77
Também alterou o
header
e oglobal.scss
.@tuliocll O que você acha ?