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

Feature/banner hidden #116

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Feature/banner hidden #116

wants to merge 7 commits into from

Conversation

PabloC5
Copy link

@PabloC5 PabloC5 commented Jun 2, 2023

COMO visitante QUERO que reduza o tamanho do painel com a escolha da cidade na tela inicial PARA eu não ver toda hora

@ronifabio ronifabio self-requested a review June 5, 2023 15:54
Copy link
Contributor

@ronifabio ronifabio left a comment

Choose a reason for hiding this comment

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

  • Não está igual a imagem do protótipo disponibilizada no Trello.
  • Colocar o nome da cidade alinhado à esquerda e ao lado o link de ALTERAR A CIDADE, quando é tamanho médio de tela. Em telas de celular, colocar a cidade à esquerda e ALTERAR CIDADE à direita.
  • A cidade precisa ser dinâmica, tem que pegar o elemento selecionado no dropdown de cidades, não pode fixar sempre na posição 0.
  • Não pode fixar ao estado do PR.
  • Ao abrir a aplicação, teria que apresentar com o painel de escolha de cidades aberto. Só esconder se o cliente clicar no botão para esconder. Feito isso, lembrar esta ação por meio do localStorage.
  • Usar o efeito de slide do jQuery para subir ou descer o painel de escolha da cidade.
  • A imagem de fundo está aparecendo cortada.
    image

@ronifabio
Copy link
Contributor

Mudar o branch base para develop ao invés de main para fazer o merge do Pull Request.

@PabloC5 PabloC5 changed the base branch from main to develop June 13, 2023 01:52
@ronifabio
Copy link
Contributor

image

  • Realizei alguns ajustes, mas ainda não está funcionando bem.
  • Não funciona para quando seleciona outra cidade, está sobrepondo o nome.
  • Também não está lembrando a escolha da cidade já realizada, devendo salvar no localStorage. Caso já tenha alguma selecionada anteriormente, já carregar esta cidade no select e também no div que você criou.
  • Usar nomenclatura dashed-case para ids e classes no HTML

… também feito alguns ajustes na função de select city
Copy link
Contributor

@ronifabio ronifabio left a comment

Choose a reason for hiding this comment

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

  • Não funciona!
  • Salvar o id da cidade no localStorage e não a imagem. Com isso, ao já ter escolhido a cidade, consegue pegar do localstorage, seta no select, setar no div de alterar cidade e ainda setar a imagem.
  • Ademais, ao selecionar a cidade no select, não está mais fechando a imagem via efeito de cortina. Usar slideOut do JQuery.
  • Outra coisa, na primeira vez que abre o sistema e não tem a cidade salva no localstorage, deve aparecer uma cidade default, no caso é Guarapuava. Pode inclusive já setar a cidade Guarapuva como default no localStorage no carregamento da aplicação.
  • Mesmo assim, nota-se que o nome da cidade não está sendo apresentado na div com o link de ALTERAR CIDADE.
  • Na verdade, se o painel com o select e imagem estiver aparecendo, o DIV com o link ALTERAR CIDADE não deve aparecer. Ele está tendo a oportunidade de escolher a cidade, não está pensando em alterar algo que ele nem escolheu ainda.

@ronifabio
Copy link
Contributor

No Trello, no primeiro quadro, tem o login e senha para acessar o cloudinary. Assim, ao invés de usar estas URLs de terceiros, seria melhor você acessar o cloudinary via interface gráfica pelo navegador e subir as imagens das duas cidades lá. Aí teremos uma URL nossa, não dependendo de outros.

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