Plots2: Menerapkan desain baru untuk halaman Tag (perlu dipecah)

Dibuat pada 15 Mar 2019  ·  46Komentar  ·  Sumber: publiclab/plots2

Membangun #4922, kita harus bekerja untuk membangun kembali halaman di https://publiclab.org/tags mengikuti desain ini:

Screen Shot 2019-03-15 at 12 45 20 PM

Ada diskusi tentang ini di sini: https://publiclab.org/notes/edie_blues/01-09-2019/the-tags-page-mockup-let-me-know-what-you-think#c22154

Template saat ini ada di sini: https://github.com/publiclab/plots2/tree/master/app/views/tag/index.html.erb

Langkah-langkah yang dapat dipecah meliputi:

  • [x] menjadikannya 2 kolom #5100
  • [x] menambahkan https://publiclab.org/stats/graph dalam iframe ke kolom ke-2
  • [x] menambahkan tautan di bawah grafik
  • [x] Mendesain ulang bilah pencarian (https://github.com/publiclab/plots2/pull/5121)
  • [ ] mendesain ulang tabel untuk menggunakan Card ui sebagai gantinya (dimulai dengan panel, di sini: https://getbootstrap.com/docs/3.3/components/#panels) dalam sebagian templat

Saya akan senang membantu dengan yang satu ini!

HTML Ruby break-me-up design help wanted

Komentar yang paling membantu

Adakah yang bisa saya bantu di sini? Akan sangat bagus jika saya bisa berkontribusi! Terima kasih

Semua 46 komentar

@jywarren Saya ingin sekali mengerjakan masalah ini. Bisakah saya mulai dengan bagian pertama membuatnya menjadi dua kolom.

Itu akan menjadi super! Terima kasih -- dan perhatikan bahwa saat ini kami menggunakan Bootstrap 3,
dokumen di https://getbootstrap.com/docs/3.3/ -- patut dicoba!

Pada Jum, 15 Mar 2019 pukul 13:31 Lekhika Dugtal [email protected]
menulis:

@warren https://github.com/warren Saya ingin sekali mengerjakan masalah ini.
Bisakah saya mulai dengan bagian pertama membuatnya menjadi dua kolom.


Anda menerima ini karena Anda yang menulis utas.
Balas email ini secara langsung, lihat di GitHub
https://github.com/publiclab/plots2/issues/5090#issuecomment-473376431 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/AABfJ8N48ACVTtYlHFbsChYZIFpjb3dpks5vW9jfgaJpZM4b24tN
.

@jywarren Haruskah saya mendesain ulang bilah pencarian?

Tentu! Tapi Anda mungkin bisa melanjutkan dengan #5832 (saya pikir?)
sudah mulai! Terima kasih!

Pada Jumat, 15 Maret 2019 pukul 13:38 Divya Baid [email protected] menulis:

@jywarren https://github.com/jywarren Haruskah saya mendesain ulang?
bilah pencarian?


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

@jywarren Apakah ini yang dikatakan bagian pertama? Saya telah membuka PR membaginya menjadi dua bagian.
Screenshot from 2019-03-16 10-11-38

@jywarren Apakah mungkin bagi orang yang sama untuk mengerjakan bagian lain juga?
Jika ya, maka saya ingin pergi untuk bagian kedua juga. :)

@ CleverFool77 Saya ingin membantu di sini! Beri tahu saya jika Anda terjebak di mana saja, kita bisa menyelesaikannya bersama!

Saya akan meminta Anda dengan pasti @IshaGupta18 :)

Hai @IshaGupta18 ,
Saya sedang mengerjakan bagian dari menunjukkan iframe. Jadi haruskah saya menulis kode lagi bersama ajax atau haruskah saya menunjukkan grafik yang dihasilkan di graph.html di sini. Saya belum pernah menggunakan Ruby on Rails pada proyek yang lengkap. Jadi masih belajar tentang ini.

Ya, saya pikir banyak orang dapat bekerja secara paralel dan/atau berkolaborasi!

Untuk iframe, kami sudah menunjukkan grafik seperti yang kami inginkan di /stats/graph, jadi
Saya pikir kita bisa memasukkannya menggunakan iframe alih-alih ajax. Apakah itu?
masuk akal?

Pada Minggu, 17 Mar 2019 pukul 07.30 Lekhika Dugtal [email protected]
menulis:

Hai @IshaGupta18 https://github.com/IshaGupta18 ,
Saya sedang mengerjakan bagian dari menunjukkan iframe. Jadi haruskah saya menulis kode
lagi bersama dengan ajax atau haruskah saya menunjukkan grafik yang dihasilkan di graph.html
di sini.


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

@jywarren Sesuatu seperti ini -
Screenshot from 2019-03-18 23-57-57

Saya belum memberikan margin. Dan grafik tag tidak terpotong, saya memperbesarnya karena tidak terlihat karena saya memiliki 3-4 grafik kecil yang berbeda. Jadi untuk saat ini saya memperbesarnya

Adakah yang bisa saya bantu di sini? Akan sangat bagus jika saya bisa berkontribusi! Terima kasih

@jywarren @gauravano Bisakah kita menggabungkan PR terlebih dahulu karena saya kira semua perubahan lain akan bergantung pada yang pertama. tautan PR
@ gautamig54 Anda dapat mulai dengan salah satu bagian dari masalah ini. Tapi sekarang bagian selanjutnya akan bergantung satu sama lain, kurasa. :pemikiran:

@ gautamig54 Saat ini, saya sedang mengerjakan grafik statistik PR untuk ditampilkan di kolom kedua. Jika Anda tertarik, Anda dapat pergi ke bagian ketiga :wink:

@CleverFool77 Oke! Saya akan mulai dengan yang ke-3! Terima kasih!

Haruskah saya menyelesaikan tugas terakhir? Bisakah saya mulai dengan bagian itu?
Terima kasih!

Ups tidak bermaksud menutup semuanya! Dibuka kembali sekarang - dan terima kasih telah berkolaborasi!

Oh, oops, ok - jadi, begitu, ada urutannya. Sepertinya @CleverFool77 telah memulai #5100 dan bisakah kita menyelesaikan hal-hal iframe di sana sebelum menggabungkannya (biasanya 2 PR akan bagus, tetapi ini sangat terikat satu sama lain -- tidak masuk akal bagi saya untuk menggabungkannya yang pertama dan kemudian menerbitkan halaman yang setengah kosong, Anda tahu?) jadi kita bisa menggabungkan halaman dasar sekaligus, lalu kita bisa melanjutkan ke beberapa halaman berikutnya? Terima kasih semuanya, ini keren!

Jika kami ingin menambahkan item baru yang 'memperbesar grafik tag', saya pikir mungkin menggunakan dokumentasi ini: http://js.cytoscape.org/

dan kode ini? -- https://github.com/publiclab/plots2/blob/master/app/views/tag/graph.html.erb

@jywarren Saya akan menambahkan iframe di pr yang sama dengan komit lain.
Menag ulang fitur zooming, itu diperlukan karena grafik lokal saya tidak memiliki banyak data.
Terima kasih

@divyabaid16 Saya kira kita perlu putus di bagian terakhir. :berpikir: Meskipun tidak yakin

Ya, Anda benar sekali @CleverFool77.
Jadi yang akan saya lakukan adalah mengubah tabel menjadi ui kartu (karena saya telah melakukan perubahan) dan mengirim PR untuk itu. Dan kemudian kami dapat bekerja sama dengan penambahan lebih lanjut di UI. Kedengarannya bagus?
Terima kasih!

Itu bagus @divyabaid16 :wink:

Oke jadi saya sangat menyesal saya tidak bisa membalas Anda @CleverFool77 . Saya akan lebih membantu dari sana. Pekerjaan luar biasa sedang dilakukan di sini! Beri tahu saya jika saya dapat membantu. Terimakasih semuanya!

Luar biasa semuanya, terima kasih!

Pada Senin, 18 Maret 2019, 23:50 Lekhika Dugtal [email protected]
menulis:

Itu bagus @divyabaid16 https://github.com/divyabaid16


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

Kami pasti akan membutuhkan bantuan Anda di masa depan @IshaGupta18 :wink:
Istirahat dan cepat sembuh @jywarren

@divyabaid16 Lakukan ping saya setelah Anda selesai dengan struktur dasar kartu :wink: . Kita akan berkolaborasi dan menyelesaikannya bersama :senyum:

Ya yakin @CleverFool77
Saya akan senang untuk bekerja dengan Anda :senyum:

Hai @jywarren Saya menemukan perbaikan yang harus dilakukan di bagian unggulan dari masalah ini. Tetapi Anda menyebutkan dalam edisi # 5004 bahwa ini disimpan dalam database. Lalu bisakah saya memiliki wawasan tentang ini?

Digabungkan #5100! Sekarang ke langkah berikutnya; mungkin https://github.com/publiclab/plots2/pull/5172 berikutnya?

Saya melihat kemajuan di https://github.com/publiclab/plots2/pull/5169 juga pada tata letak kartu!

Mari pastikan tata letak kartu oleh @CleverFool77 kompatibel dengan penyortiran di https://github.com/publiclab/plots2/pull/5176 oleh @divyabaid16 ! Kerja sama multiperson yang luar biasa di sini!!! 🙌

Hai @jywarren Tata letak kartu dasar adalah untuk bagian orang. Mengenai tata letak kartu dasar di bagian tag, saya kira @divyabaid16 berfungsi seperti yang telah dia sebutkan. Dia melakukan pekerjaan yang bagus di bagian sortir, jadi jika orang yang sama melakukan keduanya maka kompatibilitasnya akan diperiksa dengan mudah.

Nantinya, UI akan diperbaiki dari struktur dasar dengan kolaborasi antar berbagai kontributor.
Terima kasih !!!

Hai @gautamig54 , Apakah Anda butuh bantuan ?

@ CleverFool77 Terima kasih! Saya sedang mengerjakan tugas ke-3. Saya pasti akan meminta bantuan jika perlu 👍

Saya sebenarnya bingung apakah harus mengirim PR untuk kartu ui atau tidak. Karena ini adalah penerus dari 2 PR saya sebelumnya.

@jywarren @CleverFool77 Saya telah melakukan tugas ke-3 dan telah mengirimkan PR untuk hal yang sama. Tolong tinjau.

Pada halaman tag, jumlah tag yang ditampilkan di sebelah kiri lebih banyak sehingga menghasilkan ruang kosong di sebelah kanan, di bawah grafik. Saya pikir beberapa amandemen harus dilakukan mengenai hal ini dalam desain. Apa kata @jywarren @CleverFool77?

Screen Shot 2019-03-21 at 7 58 16 PM

HI @divyabaid16 saya mau tanya yg mana dua PR sebelumnya. Bukankah itu salah satu PR yang untuk sortir?

Oh saya mengerti sekarang tentang apa yang Anda maksud tentang 2 PR, @divyabaid16. Kurasa kita harus menunggu. :pemikiran:
@ gautamig54 Mari kita lihat apa yang ada di UI baru untuk ini.

@CleverFool77 Topik unggulan belum ditambahkan ke desain UI baru?

@gautmig54 Ya itu belum ditambahkan. Saya bertanya kepada @jywarren tentang pertanyaan ini di komentar di atas.
Terima kasih :)

Ah, maaf, baru merespons:

Hai @jywarren Saya menemukan perbaikan yang harus dilakukan di bagian unggulan dari masalah ini. Tetapi Anda menyebutkan dalam edisi # 5004 bahwa ini disimpan dalam database. Lalu bisakah saya memiliki wawasan tentang ini?

Ya, saya pikir kami baik-baik saja dalam hal ini sekarang: https://publiclab.org/tags

Meskipun kita harus mengganti nama halaman menjadi "Tag" dan mungkin kita dapat menghapus teksnya (saya dapat menyalinnya ke dalam database sehingga kita dapat mengubahnya dengan lebih mudah daripada melacaknya di basis kode!)

Satu hal adalah bahwa judul tabel agak ramai - saya kira kita akan menghapusnya saat kita membuat kartu?

image

Melihat https://github.com/publiclab/plots2/pull/5176/ sekarang untuk penyortiran, lalu siap untuk beralih ke tampilan kartu! ( @CleverFool77 saya melihat catatan Anda di desain kartu halaman orang, Anda benar sekali, terima kasih!)

https://github.com/publiclab/plots2/pull/5239 selesai - sekarang hanya UI kartu!

Saya akan mengirimkan PR untuk bagian ini hari ini.
Terima kasih!

Apakah halaman ini membantu?
0 / 5 - 0 peringkat