vitorbaptista / escola-que-queremos

Other
14 stars 5 forks source link

Mostrar a cidade junto ao nome da escola na filtragem dos nomes #9

Open decko opened 11 years ago

decko commented 11 years ago

Caso comum no Brasil, o nome de diversas escolas é uma homenagem a figuras publicas brasileiras. Logo, é fácil encontrar diversas escolas com o mesmo nome. Seria bacana incluir o nome da cidade na "janela" de resultados durante a filtragem do nome da escola. Ajudaria a saber qual escola estou selecionando.

escolaquequeremos-portinari

tanob commented 11 years ago

Acho que é uma boa ideia, senti a mesma dificuldade e recebi o mesmo feedback de outros dois amigos.

vitorbaptista commented 11 years ago

Curti a ideia também, @decko. A gente tinha pensado nisso durante o hackathon, mas acabou não implementando por falta de tempo. Acho que dá pra fazer tranquilo com essa funcionalidade do jquery-ui autocomplete: http://jqueryui.com/autocomplete/#custom-data.

Se você, ou outra pessoa, quiser tentar, acho do caralho :-)

vitorbaptista commented 11 years ago

Ah... e, colocando isso, era bom modificar o código para ordenar por estado e nome (ou vice-versa)... ou não, tem que ver como fica melhor.

camiloribeiro commented 11 years ago

Feito :+1: Screen Shot 2013-04-19 at 6 10 07 AM

vitorbaptista commented 11 years ago

Opa! O negócio aqui tá rápido :P

@camiloribeiro Tinha pensado em colocar o Estado, pq tem cidade com nome muito grande... e também alinhar a direita, ficando tipo:

EMEF DOM JOAO VI                                PB
EMEF JOAO RIBEIRO SOARES                        RO
EEEF JOAO FRANCISCO CORREIA                     PE

Assim acho que só com o http://jqueryui.com/autocomplete/#custom-data que passei mesmo. Vou mudar teu commit pra usar o Estado, e deixar a issue em aberto pra quem conseguir melhorar o visual, alinhando a direita. Quem sabe até colocando negrito, sei lá... belê?

vitorbaptista commented 11 years ago

Tá em produção \o/

vitorbaptista commented 11 years ago

@camiloribeiro Eu tô colocando essa issue pra você, já que vc começou fazendo :-)

Se não rolar, se "desassigna"

tanob commented 11 years ago

Minha opinião nesse issue é que deveríamos mostrar o nome da cidade e estado.

Ontem mesmo estava pesquisando uma escola e tinha que ficar escolhendo elas para ver qual a cidade.

Acho que fica legal usar aquele custom data do jquery autocomplete. On Apr 24, 2013 8:34 AM, "Vitor Baptista" notifications@github.com wrote:

@camiloribeiro https://github.com/camiloribeiro Eu tô colocando essa issue pra você, já que vc começou fazendo :-)

Se não rolar, se "desassigna"

— Reply to this email directly or view it on GitHubhttps://github.com/vitorbaptista/escola-que-queremos/issues/9#issuecomment-16921165 .

camiloribeiro commented 11 years ago

é pra voltar para a cidade? :$

tanob commented 11 years ago

Não foi isso que eu falei :-) On Apr 24, 2013 11:47 AM, "Camilo Ribeiro" notifications@github.com wrote:

é pra voltar para a cidade? :$

— Reply to this email directly or view it on GitHubhttps://github.com/vitorbaptista/escola-que-queremos/issues/9#issuecomment-16935489 .

vitorbaptista commented 11 years ago

@tanob Tá pensando em algo assim?

cidades

Acho até interessante, mas não sei... talvez fique muito repetitivo (e muito longo). Não sei quantas cidades têm o mesmo nome no mesmo estado. Você teve problemas com isso?

De qualquer forma, é melhor do que está agora...

decko commented 11 years ago

Eu pensei até em algo parecido com o exemplo de código do jQuery que o @vitorbaptista indicou, ficaria tipo

Nome da Escola em CIDADE - UF

Nome da escola em destaque, lógico

2013/4/24 Vitor Baptista notifications@github.com

@tanob https://github.com/tanob Tá pensando em algo assim?

[image: cidades]https://f.cloud.github.com/assets/76945/421145/244585a6-ad01-11e2-8e6f-f4a91f1763d6.png

Acho até interessante, mas não sei... talvez fique muito repetitivo (e muito longo). Não sei quantas cidades têm o mesmo nome no mesmo estado. Você teve problemas com isso?

De qualquer forma, é melhor do que está agora...

— Reply to this email directly or view it on GitHubhttps://github.com/vitorbaptista/escola-que-queremos/issues/9#issuecomment-16948864 .

André Filipe de Assunção e Brito Graduando em Oceanografia Centro de Estudos do Mar / UFPR twitter.com/decko

vitorbaptista commented 11 years ago

@decko Meu problema com isso é que tem nomes de cidade muito grandes, tipo "Jaboatão dos Guararapes", "Caiçara do Rio dos Ventos", etc. E, com o nome das escolas que já não são pequenos, acho que ficaria feio. Por isso pensei em só colocar o Estado.

Mas daí se ocorrer o caso que o @tanob falou, de cidades com mesmo nome e mesmo estado, então poderíamos pensar noutra solução, como a sugerida pelo Adriano.

Acho que o negócio é escolher sua preferida, implementar e ver no que dá :smile:

camiloribeiro commented 11 years ago

Gostei da solução mostrando o dropdown com o nome da cidade e estado embaixo da escola. @vitorbaptista acha que da muito trabalho?

vitorbaptista commented 11 years ago

@camiloribeiro Nada, é tranquilo. Olha lá o link pro jquery autocomplete que ele mostra um exemplo. Em 24/04/2013 23:33, "Camilo Ribeiro" notifications@github.com escreveu:

Gostei da solução mostrando o dropdown com o nome da cidade e estado embaixo da escola. @vitorbaptista https://github.com/vitorbaptista acha que da muito trabalho?

— Reply to this email directly or view it on GitHubhttps://github.com/vitorbaptista/escola-que-queremos/issues/9#issuecomment-16985597 .

vitorbaptista commented 11 years ago

@camiloribeiro Tu ainda consegue fazer essa feature?

camiloribeiro commented 11 years ago

Retomando hoje!

vitorbaptista commented 9 years ago

@camiloribeiro Te tirei dessa issue pra caso alguém queira implementá-la (vai que, né). Se você ainda tiver interesse, só avisar que coloco de volta :+1: