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

検索機能の実装 #20

Merged
merged 13 commits into from
Jun 29, 2018
Merged

検索機能の実装 #20

merged 13 commits into from
Jun 29, 2018

Conversation

kobakazu0429
Copy link
Member

@kobakazu0429 kobakazu0429 commented Apr 12, 2018

closed #19

WHY

検索機能がないと地図じゃない!

WHAT

やったこと

  • ol-extのカスタマイズ
  • 検索候補のcss
  • @aidmiya のcssの修正

やってないこと

  • 検索範囲や検索方法のカスタマイズができないので、地図データ完成後に実装したい

@kobakazu0429
Copy link
Member Author

@aidmiya レビューしてー

@kobakazu0429 kobakazu0429 changed the title empty commit 検索機能の実装 Jun 21, 2018
@aidmiya
Copy link
Collaborator

aidmiya commented Jun 23, 2018

使用感

検索機能について

  • 全体的にはとても使いやすい
  • PCで、キーワードのIMEでの変換確定時にすぐ移動するのは良くないかも

そのほか

  • 以前からの課題であるブラウザ間での表示の差異が残っている
  1. メニューボタンの大きさ(ol-extで実装すれば解消されるかと思う)
  2. ズームスライダーの可動範囲(現在はmap.cssで位置指定している状況のようですな)
  • 問題ないのかもしれないが、フルスクリーン表示にしたとき画面の下1/3ほどが黒い(何もない)

コード

ぱっと見だけど、問題ないと思うよ。
細かいところを修正してくれてありがとう。

@kobakazu0429
Copy link
Member Author

@aidmiya さんく
そのほかで上げた問題についてできればスクショなりで上げてくれると嬉しい!!

@aidmiya
Copy link
Collaborator

aidmiya commented Jun 24, 2018

ホイホイ、スクリーンショット集です。

Firefox

  1. Firefoxでは小さいメニューボタン
  2. メニューを開いている時も小さい
  3. ズームスライダーの可動範囲
  4. 実際にズームスライダーを一番上までドラッグしてみた
    image
    image
    image
    image

Chromium

  1. Chromiumではどのボタンも同じ大きさ
  2. ズームスライダーの可動範囲はFirefoxと同じくずれている
  3. 実際にズームスライダーを一番上までドラッグしてみた
    image
    image
    image

@kobakazu0429 kobakazu0429 removed the WIP label Jun 25, 2018
@kobakazu0429
Copy link
Member Author

kobakazu0429 commented Jun 29, 2018

ボタンの大きさについては #24 で修正します
ズームスライダーは #25 で修正します

@kobakazu0429 kobakazu0429 merged commit 7b7be81 into feature/dev Jun 29, 2018
@kobakazu0429 kobakazu0429 deleted the feature/search branch July 11, 2018 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

検索機能を実装する
2 participants