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

[WIP]Select query strings on search box when focused, and scroll to top when keyboard shortcut s pressed #82

Merged
merged 2 commits into from Jul 10, 2016

Conversation

kyoshidajp
Copy link
Contributor

web インターフェースに次の修正を入れてみました。

  • 検索ボックスにフォーカスが移ると、検索ボックスのキーワードをすべて選択状態にする
  • ショートカットキーの s が押されると、ページトップにスクロールする
    [cdweb] milkode.js: keep query text selected after searching #40@ongaeshi さんがコメントされているように、キーワードを継ぎ足すよりも差し替えることの方が多いと思います。これらの修正によって、より検索しやすくなると思います。

@kyoshidajp
Copy link
Contributor Author

Ruby 1.9.3のビルドが失敗していますね。json の必須 Ruby バージョンが2.0以降であるためだと思います。

対応として、Ruby のバージョンが2.0より前の場合は json 1.8.3 を参照するように Gemfile に分岐を入れるか、サポートも切れているので Travis で 1.9.3 のビルドを行わないようにするのはどうでしょうか。

@ongaeshi ongaeshi merged commit 21bfbaa into ongaeshi:master Jul 10, 2016
@ongaeshi
Copy link
Owner

ongaeshi commented Jul 10, 2016

マージリクエストありがとうございます。

  • ショートカットキーの s が押されると、ページトップにスクロールする
    • 本当はそうしたかったのですができなかった部分でした!ありがとうございます!
  • 検索ボックスにフォーカスが移ると、検索ボックスのキーワードをすべて選択状態にする
    • こちら、割と意見が分かれるところで私自身も一旦 [cdweb] milkode.js: keep query text selected after searching #40 でそのように変えたのちに別のタンミングで戻した経緯があったりします
    • が、今回のsショートカットの改善と組み合わせると 検索したいキーワードをコピー -> s -> Ctrl+v -> Enter ですぐに再検索できるのがとても便利だと感じたので選択するように戻しました
  • Travis のエラーは1.9.3のサポートを切って現在の最新版でテストするようにしました
  • 08bae48
  • https://travis-ci.org/ongaeshi/milkode

@kyoshidajp kyoshidajp deleted the feature/select-query branch August 6, 2016 08:34
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.

None yet

2 participants