Plots2: パン/ズーム時に表示されたインラインマップの表示データを更新する機能

作成日 2018年01月08日  ·  20コメント  ·  ソース: publiclab/plots2

問題(またはアイデア)を説明してください

このインラインパワータグを使用してインラインマップを埋め込むことができます

これを駆動するコードはここにあります:

https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124

そしてテンプレートはここにあります: https

ただし、マップをドラッグしても、新しい表示領域のマーカーは更新されません。

これを3つに分けましょう:

  1. [x] https://publiclab.org/api/srch/notes?srchString=fooに似ていhttps://publiclab.org/api/srch/notes?geoのように、ポイントの近くでデータのAPI呼び出しを行いましょうhttps://github.com/publiclab/plots2/blob/master/app/api/srch/search.rb#L52-L72から構築するか、新しいメソッドを作成することができます
  2. [x] https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124を再配線して、白地図を読み込み、中心点のAPI呼び出しを読み込みます。 APIからの返信のポイントとしてメモを表示します
  3. [x]マップがパンまたはズームされるたびに、中心点の近くでノートを再フェッチするようにインラインマップを設定しましょう

これは、3つ以上のステップで解決できる大きなマルチパートの問題です。 最初のものを分解することに興味がある人はいますか?

JavaScript Ruby break-me-up enhancement help wanted

最も参考になるコメント

素晴らしいプラグイン-はい、完璧です!

2018年3月30日金曜日午後4時7分、Sagarpreet Chadha < [email protected]

書きました:

@jywarren https://github.com/jywarren、@Gauravano
https://github.com/Gauravano ! この問題も解決できますか?
ライブラリを作成することにした場合、コードは後で使用される可能性があります。
どう思いますか ?


あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/publiclab/plots2/issues/1934#issuecomment-377611018
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/AABfJ9bk4gUyLKBbJxS-Qj76NBQY8i2bks5tjpCUgaJpZM4RXAwK

全てのコメント20件

@jywarrenこれを試してみたい

こんにちは、これは大きなプロジェクトですが、私はあなたがそれに取り組むのを手伝ってあげたいです
ピース。 おかげで、PRを後でではなく早く開いてください! ありがとう!

火には、2018年1月9日4:38で、のGaurav Sachdeva [email protected]
書きました:

@jywarrenhttps ://github.com/jywarrenこれを試してみたい


あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/publiclab/plots2/issues/1934#issuecomment-356231179
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/AABfJznVKoASY6PwMeUjiG4l7JCpM8vwks5tIzOmgaJpZM4RXAwK

こんにちは@jywarren 、私はあなたが述べたようにすべてのステップを検討していました。私たちはフローを次のように変更できると思います-近くのポイントのAPIを呼び出す代わりに、 https:// githubを変更することはできません

これについてどう思う?

うーん、あなたは間違いなく私たちが望むコードセクションを強調しました。 私たちは
これのために別の機能を作ることができます(多分これはあなたが言っていることです)
「Node.near()」などのように、ここで呼び出しますが、方法も提供します
クライアント側もそれを呼び出すことができます。 しかし、それを外部と呼ぶこの最後の方法
APIを介して行う必要があるようです。これは、よりユニバーサルなインターフェイスです。
他の人に使用を申し出る。 あなたはどう思いますか?

2018年1月10日17:38、「GauravSachdeva」 [email protected]は次のように書いています。

こんにちは@jywarrenhttps ://github.com/jywarren 、私はちょうどすべてを検討していました
あなたが述べた手順。私は(わからない)私たちは次のように流れを変えることができると思います-
ニアポイントのAPIを呼び出す代わりに、変更することはできません
https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df
2838528522 / app / models / concerns / node_shared.rb#L102-L124など
私たちから中心点を受け取り、近くの点を毎回提供します
パン/ズームが検出された場合、関数を呼び出します。インラインタグは次のようになります。
私たちが
ページをロードする必要があり、それ以外の場合は中央に配置します
機能する座標。

これについてどう思う?


あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/publiclab/plots2/issues/1934#issuecomment-356761050
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/AABfJ7Dnhw745kz9E6P67nx_q16MihNpks5tJTu0gaJpZM4RXAwK

うーん... APIを作成することは、この機能の将来の影響に間違いなく役立つ可能性があるので、わかりました、私はあなたの上記のフローに固執し、最初の部分のためにすぐにプルを開きます。また、plzは#1935の変更を提案します。ありがとうございます。

こんにちはジェフ私は座標をフェッチするためにhttps://github.com/publiclab/plots2/blob/01fac7cda6ab3df0ee9ba492f489edeef741ece9/app/models/concerns/node_shared.rb#L122-L146を使用しました。 /github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124は、現在使用されていないDrupalNodeCommunityTagモデルを使用しています

ああ、本当-ありがとう! それ以来、それを更新しました。

@jywarren@Gauravano 、私はこの問題の2番目と3番目の部分を行うことができるかどうか疑問に思っていましたか? これはエキサイティングです! 😄

最初の作業が終わっている間に2日から作業を開始できると思います。
PR付き; 多分@Gauravanoは最後の部分を取りたいですか?

ありがとう! マップを検索するための検索ボックスについても4番目の部分があります
ズームする追加の場所。

2018年1月13日15:18、「SagarpreetChadha」 [email protected]
書きました:

@jywarrenhttps ://github.com/jywarrenおよび@Gauravano
https://github.com/gauravano、2番目と
この問題の第3部? これはエキサイティングです! 😄


あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/publiclab/plots2/issues/1934#issuecomment-357464375
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/AABfJ1UXCiij_f-iYkgYLuypqD_FHr19ks5tKQ9egaJpZM4RXAwK

さて、2番目のものに取り組んでいます。 ありがとう😄。

@jywarren@Gauravanoさん、新しいAPIのJSON形式は次のとおりです。

screen shot 2018-01-16 at 6 46 45 pm

マップにマーカーを追加するには、各ノードの緯度と経度の値も必要に
@Gauravano APIでも各ノードの緯度と経度の値を追加できますか? ありがとう😄!

実際、 @ jywarrenは、私が構築したAPIがすべてのノードをフェッチしていることを今指摘しましたが、wikiページの誰かが[map: tag:infragramlat:lon ]のようなインラインタグを使用した場合はどうなりますか?つまり、メモ用とタグ付きメモ用の2つの異なるURLが必要です。どう思いますか?

良い呼びかけ-あなたは正しいと思いますが、最初のより単純なタイプのクエリから実装を開始し、それがどのように実行されるかを確認してから、タグベースのクエリの後続タスクを作成できます。

(更新可能なマップの新しいビューを分割する必要がある場合は、それで問題ありません。非常によく似た動作をする限り、より単純な実装を存在させることをお勧めします!)

モジュール性! :-)

わかりました、@ sagarpreet-chadhaあなたの要件に従って、私は今のところAPIを更新しています

OK、APIステップが完了してマージされました! 残りの部分を再開します:-)

このプラグインを使用できます: http地図上の場所ここのコメントからアイデアを得@jywarrenはどう思い

@ jywarren@ Gauravano ! この問題も解決できますか?
ライブラリを作成することにした場合、コードは後で使用される可能性があります。
どう思いますか ?

素晴らしいプラグイン-はい、完璧です!

2018年3月30日金曜日午後4時7分、Sagarpreet Chadha < [email protected]

書きました:

@jywarren https://github.com/jywarren、@Gauravano
https://github.com/Gauravano ! この問題も解決できますか?
ライブラリを作成することにした場合、コードは後で使用される可能性があります。
どう思いますか ?


あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/publiclab/plots2/issues/1934#issuecomment-377611018
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/AABfJ9bk4gUyLKBbJxS-Qj76NBQY8i2bks5tjpCUgaJpZM4RXAwK

これを取り戻すのは本当に素晴らしいことです!

https://publiclab.org/wiki/inline-maps

#4382で解決

このページは役に立ちましたか?
0 / 5 - 0 評価