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: criando a seção principal que contém o mapa do IFPB-CG #1

Open
wants to merge 29 commits into
base: master
Choose a base branch
from

Conversation

jpalvesl
Copy link
Member

@jpalvesl jpalvesl commented Feb 7, 2020

No description provided.

Copy link
Member

@henriquepw henriquepw left a comment

Choose a reason for hiding this comment

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

  • Cria o header de acordo com a interface criada.
  • Muda essa SecãoPrincipal para o index

Judenilson and others added 5 commits February 14, 2020 12:47
alem de separar em componentes separa sua estilizaçao tambem
a cor do icone de busca agora se altera para que não fique invisivel na caixa de pesquisa
configurando corretamente o import-helpers no eslint para organizar os imports dos components
@henriquepw
Copy link
Member

Atualizei o eslint para ele poder organizar as importações da pasta components corretamente.

svg do icone usado na barra de pesquisa adicionado
@henriquepw
Copy link
Member

Adicionei também o .svg do ícone correto da barra de pesquisa

@henriquepw henriquepw removed the request for review from rodolfobolconte February 28, 2020 00:02
Copy link
Member

@henriquepw henriquepw left a comment

Choose a reason for hiding this comment

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

  • Usar o styled-components
  • Atualizar o arquivo themes.js com as cores corretas
  • Atualizar o Header de acordo com o design feito

💡 dica: o arquivo com o design no figma já da várias informações para criar o css, alias, se clicar em algum componente, ele já da todo o css, mas não use ele completo, use mais para saber os espaçamentos.

jpalvesl and others added 10 commits February 29, 2020 00:03
usando o svg-inline-react é colocado o svg na seção principal e usando o mousehover para alterar o background de cada sala ou bloco do svg
Alteração dos campos vazios trocando separador e palavras em maiúscula.
remove o pacote anteriormente utilizado para usar os svg's e adiciona o plugin do gatsby para isso
remove a estilização presente em Layout referente ao footer e agora ela se encontra no styled component FooterSC
remove as dependencias no componente HeaderSC referente a caixa de pesquisa e cria um componente
separado para isso
@henriquepw
Copy link
Member

No header, deixa a lupa clicável e usa um state para dizer se ele ta aberto ou fechado.

Presta atenção no modelo do Figma, os ícones ficão juntos da direita, em vez da lupa no meio, se você selecionar um componente no Figma, segura Alt e bota o mouse em cima de outro, ele diz a distancia entre eles.

Screenshot from 2020-03-21 10-28-10

Tem a parte de code do Figma tbm, que ajuda.

Screenshot from 2020-03-21 10-26-43

alem de posicionar os icones, agora a lupa também é clicavel e quando esse evento ocorro ou a tecla
enter é pressionada na caixa de pesquisa uam função é chamada
@jpalvesl jpalvesl force-pushed the feat/criar-seção-principal branch from 78dad8e to 8450dfb Compare March 23, 2020 05:59
@jpalvesl
Copy link
Member Author

Posicionei os icones de acordo com o Wireframe, porém mesmo com a largura 0 o input continua aparecendo antes de clicar na lupa(da pra ver até pelo mouse se a pessoa for com ele a esquerda da lupa), eu até tentei usar um useState pra deixar "display: none" e depois quando eu clicasse na lupa fosse pra "display: block", mas isso faz com que não tenha animação

@henriquepw
Copy link
Member

  • Ao clicar novamente na lupa, eu não consigo fechar o a caixa de pesquisa, ao invés disso ele abre um alert:
    Alert

-A janela flutuante ficaria melhor se fosse com opacidade 1, do jeito que ta dificulta a leitura.

@jpalvesl jpalvesl force-pushed the feat/criar-seção-principal branch from 44e1dcc to 96c3f39 Compare April 6, 2020 19:45
@jpalvesl jpalvesl force-pushed the feat/criar-seção-principal branch from 036b1d7 to bbcce5a Compare August 11, 2020 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants