Pushpin: Batas kartu pengaturan FileContent sama untuk setiap jenis file

Dibuat pada 13 Feb 2020  ·  7Komentar  ·  Sumber: automerge/pushpin

Berdasarkan tampilannya, AudioContent seharusnya benar-benar memiliki tinggi minimal 3, tetapi sepertinya FileContent menyetelnya ke 6. Apakah masuk akal untuk menulis ulang FileContent agar memiliki batasan yang berbeda untuk file yang berbeda jenis? Tampaknya sepele untuk menulis logika itu di dalam FileContent.tsx, tetapi tampaknya lebih baik untuk mengatur properti tersebut di setiap file [mime-type]Content.tsx . Bagaimana saya akan melakukan ini? Maaf masih bingung bagaimana data mengalir antara FileContent dan AudioContent .

Komentar yang paling membantu

Maksud saya, bisakah kita membuat komentar menjadi tipe arbitrer, seperti ThreadContent, TextContent, ImageContent, dll.

(PS: jika Anda belum membaca ARCHITECTURE.md, mungkin akan mempercepat pemahaman Anda tentang cara kerja sistem di belakang layar.)

Semua 7 komentar

Ah ya. Ini sedikit bodoh dan pasti salahku. AudioContent adalah POC yang saya tulis yang tidak benar-benar dimaksudkan untuk mendarat di master tetapi kami bergabung ketika kami menguji peningkatan pada dukungan streaming file biner.

Masalahnya adalah FileContent pada dasarnya membungkus AudioContent dengan melihat bidang mimetype dari hyperfile dan kemudian memilih jenis Konten berdasarkan itu (ikuti https://github.com/automerge/pushpin/blob/7b1fb02912198c525bf080edd5d14d48e538b729/src/renderer/components/ content-types/files/FileContent.tsx#L72 jika Anda ingin melihat cara kerjanya). Itu berarti selalu menggunakan minWidth pembungkus FileContent dalam praktiknya.

Memperbaiki ini tidak rumit tetapi tidak terlalu sepele dan saya tidak meyakinkan diri sendiri bahwa ada orang yang benar-benar menggunakan fungsi ini sejak awal jadi saya tidak menyiasatinya... Salah satu opsi, alih-alih memperbaikinya, adalah memasukkan sedikit metadata ekstra / pilih set elemen UI yang lebih besar sehingga mengisi ruang? Kecurangan, saya tahu, tapi saya ingin mulai menyiarkan posisi Anda di file audio seperti kami menyiarkan set kartu yang dipilih di papan sehingga Anda dapat melihat di mana orang lain berada di trek (berguna untuk, katakanlah, beberapa kasus penggunaan podcast imajiner?)

Senang untuk menggulir dengan salah satu pendekatan (memperbaiki bug pengaturan ketinggian atau meningkatkan AudioContent) dan membantu jika Anda ingin mengatasinya. Harus menjadi proyek kecil yang bagus.

Menambahkan metadata/penyiaran ekstra terdengar seperti hal yang keren untuk dikerjakan! File apa yang harus saya cari untuk logika penyiaran pemilihan kartu?

Oooo juga bagaimana perasaan Anda tentang komentar bergaya soundcloud yang terkait dengan waktu tertentu dalam audio? Atau apakah itu menduplikasi terlalu banyak fungsi utas? Saya pikir komentar gaya khusus dokumen misalnya seperti ini (atau komentar pada gambar dengan koordinat XY seperti tag instagram) dapat mengaktifkan jenis percakapan khusus media yang menarik.

Saya akan merasa seperti neraka ya . Komentar berkode waktu jelas harus disimpan dalam dokumen entah bagaimana... Konten sewenang-wenang? Apakah itu terlalu gila?

Untuk menyiarkan posisi mendengarkan, Anda dapat mengirim "Kehadiran" yang menyiarkan id pengguna, id perangkat, dan tipe tambahan yang dapat dikodekan JSON sewenang-wenang ke rekan-rekan Anda. Di BoardCard, Anda menyiarkan warna pilihan Anda ke rekan-rekan Anda. Kode ini agak aneh, saya tahu, tetapi lihat usePresence di https://github.com/automerge/Pushpin/blob/73193adc907b3c7c109b5f14453f9a838469f02b/src/renderer/components/content-types/board/BoardCard.tsx

Yang Anda maksud dengan 'Konten sewenang-wenang' adalah tipe Konten baru (misalnya audioComments ) dengan larik { comment: string, time: number, author: Contact} dan id hypermergeUrl dari audio yang sesuai? Atau hanya array { comment: string, time: number, author: Contact} yang dilampirkan ke AudioContent itu sendiri? Apakah melampirkan JSON sewenang-wenang ke hyperfile bahkan mungkin? Maaf saya masih belajar bagaimana arsitektur ini bekerja.

Maksud saya, bisakah kita membuat komentar menjadi tipe arbitrer, seperti ThreadContent, TextContent, ImageContent, dll.

(PS: jika Anda belum membaca ARCHITECTURE.md, mungkin akan mempercepat pemahaman Anda tentang cara kerja sistem di belakang layar.)

(memindahkan ini ke kendur karena kami telah melakukan perjalanan cukup jauh dari masalah aslinya)

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

Gozala picture Gozala  ·  13Komentar

pvh picture pvh  ·  4Komentar

Gozala picture Gozala  ·  9Komentar

edrex picture edrex  ·  7Komentar

canadaduane picture canadaduane  ·  9Komentar