Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Open
decko opened this issue Apr 18, 2013 · 18 comments
Open

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

decko opened this issue Apr 18, 2013 · 18 comments

Comments

@decko
Copy link

decko commented Apr 18, 2013

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
Copy link
Collaborator

tanob commented Apr 19, 2013

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

@vitorbaptista
Copy link
Owner

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
Copy link
Owner

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
Copy link
Collaborator

Feito 👍
Screen Shot 2013-04-19 at 6 10 07 AM

@vitorbaptista
Copy link
Owner

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 vitorbaptista reopened this Apr 19, 2013
@vitorbaptista
Copy link
Owner

Tá em produção \o/

@ghost ghost assigned camiloribeiro Apr 24, 2013
@vitorbaptista
Copy link
Owner

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

Se não rolar, se "desassigna"

@tanob
Copy link
Collaborator

tanob commented Apr 24, 2013

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//issues/9#issuecomment-16921165
.

@camiloribeiro
Copy link
Collaborator

é pra voltar para a cidade? :$

@tanob
Copy link
Collaborator

tanob commented Apr 24, 2013

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//issues/9#issuecomment-16935489
.

@vitorbaptista
Copy link
Owner

@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
Copy link
Author

decko commented Apr 24, 2013

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//issues/9#issuecomment-16948864
.

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

@vitorbaptista
Copy link
Owner

@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á 😄

@camiloribeiro
Copy link
Collaborator

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

@vitorbaptista
Copy link
Owner

@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//issues/9#issuecomment-16985597
.

@vitorbaptista
Copy link
Owner

@camiloribeiro Tu ainda consegue fazer essa feature?

@camiloribeiro
Copy link
Collaborator

Retomando hoje!

@vitorbaptista
Copy link
Owner

@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 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

4 participants