haushinka2dx / atmosphere-web

1 stars 2 forks source link

Search Timeline #62

Closed haushinka2dx closed 10 years ago

haushinka2dx commented 10 years ago

description

atmosphere-search

known issue

  1. 検索キーワードの入力欄の入力補完をやろうとすると、候補が隠れてしまうので今は適用していない
haushinka2dx commented 10 years ago

b81d8c7 なんかこれが含まれちゃってるけどうまく消せない…。

kannokanno commented 10 years ago

+1

kannokanno commented 10 years ago

あとは良さそうです

haushinka2dx commented 10 years ago

修正出来るところはしました。

逆に残っているのは以下のものです。

キーワード入力エリアで Enter すると検索できる機能を実装してます。 で、入力補完は jquery-textcomplete で補完してるんですが、ライブラリ側で keydown イベントを取ってしまっていて Enterで検索 の機能が死んでしまいます。

つまり、以下の二者択一になってしまっています。

今のところ両立の方法が見つかっていないのですが、どちらかを取るとしたらどちらが良いですかね? 意見いただけると幸いです。 @kannokanno @y-motegi @Imamotty @ip-s-pra さん(マストじゃないっす)

shunsuke-suzuki-zeroboard commented 10 years ago

投稿と動作を揃えればどちらでもいいと思います。

kannokanno commented 10 years ago

どちらかと言えばこっち。

もしくは「forkしてEnterで補完させないように修正したやつを使う」

haushinka2dx commented 10 years ago

@ip-s-pra

投稿と動作を揃えればどちらでもいいと思います。

動作を揃えるってのはありですね。 が、検索は基本的には改行が不要だと思いますが、投稿は必要だったりと微妙に要件が違うようにも思います。 ということで今回は Enterで検索 にしようと思います。

@kannokanno

もしくは「forkしてEnterで補完させないように修正したやつを使う」

確かにそれはありですね。 それにしても Enter が使えなくなるというユーザー操作への影響はあるので、ちゃんと考えた方がいいかもですね。

他になければマージしちゃいます。

kannokanno commented 10 years ago

okでーす

haushinka2dx commented 10 years ago

今のところLGTMはないと…。