Plots2: kemampuan untuk menyegarkan data yang ditampilkan peta sebaris yang ditampilkan saat Anda menggeser/memperbesar

Dibuat pada 8 Jan 2018  ·  20Komentar  ·  Sumber: publiclab/plots2

Tolong jelaskan masalah (atau ide)

Anda dapat menyematkan peta sebaris menggunakan tag daya sebaris ini: https://publiclab.org/wiki/inline-maps

Kode yang mendorong ini ada di sini:

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

Dan templatenya ada di sini: https://github.com/jywarren/plots2/blob/master/app/views/map/_leaflet.html.erb

Namun, saat Anda menyeret peta, penanda tidak disegarkan untuk area tampilan baru.

Mari kita pecahkan ini menjadi tiga:

  1. [x] mari kita membuat panggilan API untuk data di dekat suatu titik, mirip dengan https://publiclab.org/api/srch/notes?srchString=foo tapi mungkin https://publiclab.org/api/srch/notes?geo =41.0 ,-91.3. Ini dapat dibangun dari https://github.com/publiclab/plots2/blob/master/app/api/srch/search.rb#L52 -L72, atau metode baru dapat dibuat
  2. [x] mari kita rewire https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124 untuk memuat peta kosong yang kemudian memuat panggilan API untuk titik pusat, dan menampilkan catatan sebagai poin saat mendengar kembali dari API
  3. [x] mari atur peta sebaris untuk mengambil kembali catatan di dekat titik pusatnya setiap kali peta digeser atau diperbesar

Ini adalah masalah multi-bagian besar yang dapat kita selesaikan dalam tiga langkah atau lebih. Ada yang tertarik untuk membocorkan yang pertama?

JavaScript Ruby break-me-up enhancement help wanted

Komentar yang paling membantu

plugin yang luar biasa -- ya sempurna!

Pada Jumat, 30 Mar 2018 pukul 16:07, Sagarpreet Chadha < [email protected]

menulis:

@jywarren https://github.com/jywarren , @Gauravano
https://github.com/Gauravano ! Bisakah saya juga menyelesaikan masalah ini?
Kode tersebut dapat digunakan nanti jika kita memutuskan untuk membuat perpustakaan.
Bagaimana menurut anda ?


Anda menerima ini karena Anda disebutkan.
Balas email ini secara langsung, lihat di GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-377611018 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/AABfJ9bk4gUyLKBbJxS-Qj76NBQY8i2bks5tjpCUgaJpZM4RXAwK
.

Semua 20 komentar

@jywarren saya ingin mencoba yang ini

hai, ini adalah proyek besar tetapi saya akan dengan senang hati membantu Anda menanganinya sedikit demi sedikit
bagian. Terima kasih dan buka PR lebih cepat daripada nanti! Terima kasih!!

Pada Selasa, 9 Januari 2018 pukul 04.38, Gaurav Sachdeva [email protected]
menulis:

@jywarren https://github.com/jywarren Saya ingin mencoba yang ini


Anda menerima ini karena Anda disebutkan.
Balas email ini secara langsung, lihat di GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-356231179 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/AABfJznVKoASY6PwMeUjiG4l7JCpM8vwks5tIzOmgaJpZM4RXAwK
.

Hai @jywarren , saya baru saja mempertimbangkan semua langkah seperti yang Anda nyatakan. Saya pikir (tidak yakin) kita dapat mengubah aliran sebagai - alih-alih membuat panggilan ke API untuk titik dekat, tidak bisakah kita memodifikasi https://github. com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124 seperti itu akan menerima titik pusat dari kami dan memberi kami titik dekat dan setiap kali pan/zoom terdeteksi, kami akan memanggil fungsinya .Tag inline akan ditangani dengan membuat fungsi tambahan yang harus digunakan hanya ketika kita harus memuat halaman dan di lain waktu kita hanya akan memberikan koordinat pusat untuk berfungsi.

Apa yang Anda pikirkan tentang ini?

Hmm, nah Anda pasti menyorot bagian kode yang kita inginkan. Saya pikir kita
dapat membuat fungsi lain untuk ini (mungkin ini yang Anda katakan)
seperti "Node.near()" atau semacamnya, sebut saja di sini, tetapi juga berikan caranya
sisi klien dapat menyebutnya juga. Tapi cara terakhir menyebutnya secara eksternal
sepertinya itu harus melalui API, antarmuka yang lebih universal yang kita bisa
menawarkan kepada orang lain untuk digunakan. Apa yang kamu pikirkan?

Pada 10 Januari 2018 17:38, "Gaurav Sachdeva" [email protected] menulis:

Hai @jywarren https://github.com/jywarren , saya baru saja mempertimbangkan semuanya
langkah-langkah seperti yang Anda nyatakan. Saya pikir (tidak yakin) kami dapat mengubah aliran sebagai -
alih-alih membuat panggilan ke API untuk titik dekat, tidak bisakah kita memodifikasi?
https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df
2838528522/app/models/concerns/node_shared.rb#L102-L124 seperti itu akan
terima titik pusat dari kami dan berikan kami titik dekat dan setiap saat
pan/zoom terdeteksi, kita akan memanggil fungsinya. Tag inline akan menjadi
ditangani dengan membuat fungsi tambahan yang harus digunakan hanya ketika kita
harus memuat halaman dan di lain waktu kami hanya akan memberikan pusat
koordinat untuk berfungsi.

Apa yang Anda pikirkan tentang ini?


Anda menerima ini karena Anda disebutkan.
Balas email ini secara langsung, lihat di GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-356761050 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/AABfJ7Dnhw745kz9E6P67nx_q16MihNpks5tJTu0gaJpZM4RXAwK
.

Hmm...membuat API pasti bisa berguna untuk implikasi masa depan dari fitur ini jadi ok, saya akan tetap dengan alur yang Anda sebutkan di atas dan akan segera membuka tarikan untuk bagian pertama. Juga, tolong sarankan perubahan di #1935 .Terima kasih.

Hai jeff Saya telah menggunakan https://github.com/publiclab/plots2/blob/01fac7cda6ab3df0ee9ba492f489edeef741ece9/app/models/concerns/node_shared.rb#L122 -L146 untuk mengambil koordinat, fungsi yang diberikan pada tautan repo Anda yaitu, https:/ /github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124 menggunakan model DrupalNodeCommunityTag yang tidak digunakan sekarang

Aha, benar -- terima kasih! Kami memperbarui itu sejak.

@jywarren dan @Gauravano , saya ingin tahu apakah saya dapat mengerjakan bagian ke-2 dan ke-3 dari masalah ini ? Ini akan menarik!! 😄

Saya pikir Anda bisa mulai bekerja pada tanggal 2 sementara yang pertama selesai,
dengan PR; mungkin @Gauravano ingin mengambil bagian terakhir?

Terima kasih! Akan ada bagian ke-4 juga tentang kotak pencarian untuk mencari peta
lokasi tambahan untuk memperbesar.

Pada 13 Jan 2018 15:18, "Sagarpreet Chadha" [email protected]
menulis:

@jywarren https://github.com/jywarren dan @Gauravano
https://github.com/gauravano , saya ingin tahu apakah saya dapat melakukan yang ke-2 dan
3 bagian dari masalah ini? Ini akan menarik!! 😄


Anda menerima ini karena Anda disebutkan.
Balas email ini secara langsung, lihat di GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-357464375 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/AABfJ1UXCiij_f-iYkgYLuypqD_FHr19ks5tKQ9egaJpZM4RXAwK
.

Oke, bekerja pada yang kedua. Terima kasih .

Hai @jywarren dan @Gauravano , format JSON dari API baru adalah:

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

Untuk menambahkan penanda ke peta, kita juga membutuhkan nilai Latitude dan Longitude dari setiap node.
@Gauravano Bisakah Anda juga menambahkan nilai Latitude dan Longitude dari setiap node juga di API? Terima kasih !

Sebenarnya @jywarren Saya baru saja mencatat sekarang bahwa API yang saya buat mengambil semua node tetapi bagaimana jika seseorang di halaman wiki menggunakan tag inline seperti [map: tag:infragram : lat:lon ] lalu, kami tidak memiliki API untuk itu.Jadi, kita perlu memiliki 2 URL yang berbeda satu untuk catatan dan satu untuk catatan dengan tag.Bagaimana menurut Anda?Apakah saya berbicara dengan benar?

Panggilan yang bagus -- Saya rasa Anda benar, tetapi kita dapat memulai implementasi pada jenis kueri pertama yang lebih sederhana, lihat kinerjanya, lalu buat tugas lanjutan untuk yang berbasis tag.

(Jika Anda perlu membagi tampilan baru untuk peta yang dapat disegarkan, tidak apa-apa -- mungkin bagus untuk memiliki implementasi yang lebih sederhana selama itu bekerja dengan sangat mirip!)

Modularitas! :-)

Ok lalu @sagarpreet-chadha sesuai kebutuhan Anda, saya memperbarui API untuk saat ini

Oke, langkah API selesai dan digabungkan! Membuka kembali untuk bagian yang tersisa :-)

Kita dapat menggunakan plugin ini: http://labs.easyblog.it/maps/leaflet-search/ untuk mencari lokasi di peta (dapat ide dari komentar Anda di sini ). Bagaimana menurutmu @jywarren ?

@jywarren , @Gauravano ! Bisakah saya juga menyelesaikan masalah ini?
Kode tersebut dapat digunakan nanti jika kita memutuskan untuk membuat perpustakaan.
Bagaimana menurut anda ?

plugin yang luar biasa -- ya sempurna!

Pada Jumat, 30 Mar 2018 pukul 16:07, Sagarpreet Chadha < [email protected]

menulis:

@jywarren https://github.com/jywarren , @Gauravano
https://github.com/Gauravano ! Bisakah saya juga menyelesaikan masalah ini?
Kode tersebut dapat digunakan nanti jika kita memutuskan untuk membuat perpustakaan.
Bagaimana menurut anda ?


Anda menerima ini karena Anda disebutkan.
Balas email ini secara langsung, lihat di GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-377611018 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/AABfJ9bk4gUyLKBbJxS-Qj76NBQY8i2bks5tjpCUgaJpZM4RXAwK
.

Ini akan sangat luar biasa untuk diambil kembali!

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

Dipecahkan dalam #4382

Apakah halaman ini membantu?
0 / 5 - 0 peringkat