Pushpin: ترتبط بطاقة إعداد محتوى الملف بنفس الحدود لكل نوع ملف

تم إنشاؤها على ١٣ فبراير ٢٠٢٠  ·  7تعليقات  ·  مصدر: automerge/pushpin

بناءً على شاشة العرض ، يجب أن يكون الحد الأدنى للارتفاع AudioContent 3 ، ولكن يبدو أن FileContent يضبطه على 6. هل من المنطقي إعادة كتابة FileContent للحصول على حدود مختلفة لملف مختلف أنواع؟ يبدو من التافه كتابة هذا المنطق داخل FileContent.tsx ، ولكن يبدو من الأفضل تعيين تلك الخصائص في كل ملف [mime-type]Content.tsx . كيف لي أن تذهب عن القيام بذلك؟ آسف ، ما زلت مرتبكًا حول كيفية تدفق البيانات بين FileContent و AudioContent .

التعليق الأكثر فائدة

أعني ، هل يمكننا جعل التعليقات من النوع التعسفي ، مثل ThreadContent ، TextContent ، ImageContent ، إلخ.

(ملاحظة: إذا لم تكن قد قرأت ARCHITECTURE.md ، فقد يؤدي ذلك إلى تسريع فهمك لكيفية عمل النظام خلف الكواليس.)

ال 7 كومينتر

اه نعم. هذا غبي بعض الشيء وبالتأكيد خطأي. كان AudioContent عبارة عن POC كتبته ولم يكن من المفترض حقًا أن يصل إلى المستوى الرئيسي ولكننا اندمجنا عندما كنا نختبر تحسينات على دعم تدفق الملفات الثنائية.

تكمن المشكلة في أن FileContent يلتف بشكل أساسي على AudioContent من خلال النظر في حقل mimetype للملف التشعبي ثم اختيار نوع المحتوى بناءً على ذلك (اتبع https://github.com/automerge/pushpin/blob/7b1fb02912198c525bf080edd5d14d48e538b729/src/renderer/ content-types / files / FileContent.tsx # L72 إذا كنت تريد أن ترى كيف يعمل). هذا يعني أنه يستخدم دائمًا minWidth الخاص بـ FileContent في الممارسة العملية.

إصلاح هذا ليس معقدًا ولكنه ليس تافهًا حقًا ولم أقنع نفسي بأن أي شخص كان يستخدم الوظيفة بالفعل في المقام الأول لذلك لم ألتف حوله ... أحد الخيارات ، بدلاً من إصلاحه ، سيكون تضمين القليل من البيانات الوصفية الإضافية / اختيار مجموعة أكبر من عناصر واجهة المستخدم بحيث تملأ المساحة؟ الغش ، أعلم ، لكنني أريد أن أبدأ في بث الموضع الذي أنت فيه في ملف الصوت تمامًا مثلما نبث مجموعة البطاقات المحددة في لوحة حتى تتمكن من معرفة مكان وجود الأشخاص الآخرين في المسار (مفيد ، على سبيل المثال ، بعض حالات استخدام البودكاست الخيالية؟)

يسعدنا التعامل مع أي من الطريقتين (إصلاح خطأ إعداد الارتفاع أو تحسين محتوى الصوت) والمساعدة في أي منهما إذا كنت ترغب في معالجته. يجب أن يكون مشروعًا صغيرًا لطيفًا في كلتا الحالتين.

يبدو أن إضافة بيانات وصفية / إذاعة إضافية أمر رائع للعمل عليه! ما هي الملفات التي يجب أن أبحث فيها عن منطق بث اختيار البطاقة؟

Oooo أيضًا ما هو شعورك حيال التعليقات على غرار soundcloud المرتبطة بأوقات محددة في الصوت؟ أم أن هذا يؤدي إلى تكرار الكثير من وظائف سلاسل الرسائل؟ أعتقد أن التعليقات الخاصة بأسلوب المستند على سبيل المثال (أو التعليقات على الصور ذات الإحداثيات XY مثل علامات instagram) يمكن أن تتيح أنواعًا مثيرة للاهتمام من المحادثات الخاصة بالوسائط.

نعم أشعر

لبث موقع استماع ، يمكنك إرسال "حضور" الذي يبث معرف المستخدم الخاص بك ، ورقم تعريف الجهاز ، ونوع JSON الإضافي العشوائي القابل للتشفير إلى أقرانك. في BoardCard ، تقوم ببث لون التحديد الخاص بك إلى زملائك. هذا الرمز غريب بعض الشيء ، أعلم ، لكن انظر إلى usePresence في https://github.com/automerge/pushpin/blob/73193adc907b3c7c109b5f14453f9a838469f02b/src/renderer/components/content-types/board/BoardCard.content-types/board/BoardCard.

من خلال "المحتوى التعسفي" ، هل تقصد نوع محتوى جديد (مثل audioComments ) بمصفوفة من { comment: string, time: number, author: Contact} ومعرف hypermergeUrl للصوت المقابل؟ أو مجرد مصفوفة من { comment: string, time: number, author: Contact} مرفقة بـ AudioContent نفسها؟ هل يمكن إرفاق JSON التعسفي بملف مفرط؟ آسف ، ما زلت أتعلم كيف تعمل هذه الهندسة.

أعني ، هل يمكننا جعل التعليقات من النوع التعسفي ، مثل ThreadContent ، TextContent ، ImageContent ، إلخ.

(ملاحظة: إذا لم تكن قد قرأت ARCHITECTURE.md ، فقد يؤدي ذلك إلى تسريع فهمك لكيفية عمل النظام خلف الكواليس.)

(نقل هذا إلى Slack نظرًا لأننا ابتعدنا كثيرًا عن الإصدار الأصلي)

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات

القضايا ذات الصلة

canadaduane picture canadaduane  ·  9تعليقات

Gozala picture Gozala  ·  9تعليقات

Gozala picture Gozala  ·  13تعليقات

Gozala picture Gozala  ·  4تعليقات

pvh picture pvh  ·  4تعليقات