Skip to content

Conversation

@kyoya0819
Copy link
Member

Close #75

@netlify
Copy link

netlify bot commented Oct 19, 2021

Deploy Preview for funabashi-map ready!

Name Link
🔨 Latest commit bff2185
🔍 Latest deploy log https://app.netlify.com/sites/funabashi-map/deploys/62a582e7949f730009ff6968
😎 Deploy Preview https://deploy-preview-78--funabashi-map.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@j-takurou
Copy link
Contributor

j-takurou commented Oct 23, 2021

開発環境で正常に動作しない問題の修正

に関して、

自分のOSとchromeのversionの問題かわからないですが、以下の環境でエラーが発生しました。

以下のキャッシュ削除処理をしてなかっただけでした。orz

rm -rf node_modules/.cache
npm i -D
npm run start

私の環境

OS: M1 mac Big Sur version 11.3
chrome: Version 94.0.4606.71 (編集済み)

エラー文

Failed to compile.

./node_modules/@react-leaflet/core/esm/path.js 10:41
Module parse failed: Unexpected token (10:41)
File was processed with these loaders:
 * ./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|   useEffect(function updatePathOptions() {
|     if (props.pathOptions !== optionsRef.current) {
>       const options = props.pathOptions ?? {};
|       element.instance.setStyle(options);
|       optionsRef.current = options;

@aca-ta
Copy link
Member

aca-ta commented Oct 25, 2021

使い勝手に関するコメントです。
個人の感想なので、採用するかはおまかせします。

  • ナビゲータを隠すオプションが欲しい
    • スマホで地図を見るとやや圧迫感があると思いました
  • 現在地に移動する際は、現在地のマーカーが表示されて欲しい
  • 現在地へ ボタンを押すより現在地へ移動するアイコンが欲しい
    • google mapやyahoo map等と同じため、直感的と思いました

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.

マップ表示ページの表示内容(場所)改善

4 participants