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

Refeito a tela de um partido #82

Merged
2 commits merged into from
Oct 20, 2020
Merged

Refeito a tela de um partido #82

2 commits merged into from
Oct 20, 2020

Conversation

ghost
Copy link

@ghost ghost commented Oct 16, 2020

#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 o global.scss.

@tuliocll O que você acha ?

@ghost ghost changed the title refeito a tela de um partido Refeito a tela de um partido Oct 16, 2020
@tuliocll
Copy link
Contributor

Boa! Ficou excelente!
Eu também adicionei a montserrat cá, mas qualquer coisa no merge eu corrijo, inclusive depois de finalizar toda UI seria bom a gente abrir mais duas issues, uma para unificar a tipografia e outra pra margin e padding dos componentes (eu deveria ter padronizado isso no protótipo, mas acabou não dando), mas isso quando a gente finalizar.
Eu to atrasado com a minha tela mas logo logo eu solto ela, vou pegar a issue da tela de listagem de partidos, por que fiz componentes que da pra reutilizar lá, então já vai evitar o retrabalho e adiantar bastante.

@@ -15,6 +15,10 @@ header {
align-items: center;
background-color: #ffffff;
filter: drop-shadow(0px 4px 10px rgba(0, 0, 0, 0.25));

> div {
Copy link
Contributor

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;
Copy link
Contributor

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}
Copy link
Contributor

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

<h1 className="title is-4">
{partido.nome} - {partido.sigla}
<div className="divPartido">
<div className="infoPartido">
Copy link
Contributor

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/

@ghost
Copy link
Author

ghost commented Oct 17, 2020

  • Te adicionei como responsável da tela de listagem de partidos.
  • Quando tiver um tempo vou fazer essas correções.
  • Acho uma boa ideia criar essas issues para a padronização mesmo(depois que a gente finalizar tudo)

@ghost
Copy link
Author

ghost commented Oct 17, 2020

Fiz as alterações solicitadas.

@tuliocll
Copy link
Contributor

massa

@ghost ghost merged commit 301c01e into master Oct 20, 2020
This pull request was closed.
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.

2 participants