pokemongovet / pokemongovet.github.io

Projeto para incentivar adoção de animais pessoas que amam pokemons virtuais
https://pokemongovet.github.io/
Other
103 stars 38 forks source link

Página de listagem de centros cadastrados. #43

Closed cauequeiroz closed 8 years ago

cauequeiroz commented 8 years ago

Página "Adote um Pokemon de Verdade!". Pensei em fazer algo bem limpo e organizado, com a navegação no topo idealizada pelo @fititnt e todos os estados e locais estruturados.

Para estados que não tem nada cadastrado, deixei uma mensagem com link para o usuario cadastrar. Para estados que tem algo cadastrados, sugiro seguir esse padrão, não importando se é local de adoção, grupo do facebook ou ong. Obs: deletei o PR antigo por motivos tecnicos, estou enviando esse já com todos os cadastros que foram feitos até hoje atualizados para essa nova estrutura.

Esse PR é referente a um dos pontos levantados na issue #21

@fititnt

fititnt commented 8 years ago

Eu vou testar isso. Tem mais alguém alem de mim e do @cauequeiroz pra verificar e aprovar antes de darmos merge?

evansantos commented 8 years ago

@fititnt já baixo o pr e dou uma olhada e falo se está ok, beleza?

fititnt commented 8 years ago

@evansantos perfeito

evansantos commented 8 years ago

@cauequeiroz @fititnt LGTM 👍 Só um detalhe a separação por cidade está faltando, acho que corrigindo isso pode mandar bala, Emerson


EDIT:

Lendo a w3c vi que você fez varias listas na separação dos estados no começo do html Caue, você poderia utilizar o atributo start na ul

cauequeiroz commented 8 years ago

@evansantos Opa cara, obrigado por conferir! Quando a separação por cidade, foi intencional mesmo. Como ainda não temos muitos dados por cidade, iria ficar pesado visualmente varias "seções" para cada uma ter um dado apenas, ou apenas uma "seção" para o estado todo. Não vejo problemas em um primeiro momento separarmos apenas por estados, e conforme a lista for crescendo, vamos evoluindo isso. O que acha?

Sobre a separação em varias listas, foi simplesmente para criar 4 colunas mantendo tudo em ordem alfabetica. Pode explicar um pouco melhor onde esse atributo entraria nesse caso?

fititnt commented 8 years ago

Testei aqui. Por mim está melhor o resultado, porém visualmente ficou mais pesado e ocupando mais espaço. Agora tudo ficou em um estado, e a cidade é uma propriedade do local. O mínimo que poderia fazer é um hotfix pra reduzir o tamanho da fonte dos titulos que são os links para as organizações que são facebook apenas.

fititnt commented 8 years ago

Se você estiver meio na pressa ou for dormir logo, a gente aceita e você faz um PR melhorado depois.

cauequeiroz commented 8 years ago

@fititnt algo nesse estilo? Diminui fonte e alguns espaçamentos...

fititnt commented 8 years ago

@cauequeiroz eu vou liberar seu PR aqui e depois ou abre outro PR ou eu mesmo edito ali no master.

@pokemongovet/lideres-de-ginasio podem aprovar esse PR aqui?

evansantos commented 8 years ago

vou checar aqui 😄

rcgivisiez commented 8 years ago

Obrigada pelo PR, @cauequeiroz! ;)

fititnt commented 8 years ago

Eita, ta todo mundo online e respondendo rápido

evansantos commented 8 years ago

LGTM 👍