jack2
20 ديسمبر 2019، 11:05ص
1
كيفية إعادة إنتاج المشكلة:
انتقل إلى أحد منشوراتك وانقر على “تحرير” لفتح المحرر. لا تكتب أي شيء.
اضغط على مفتاح Esc => كما هو متوقع، يختفي المحرر تمامًا (لأنه لم يتم تصغيره، نظرًا لأن المنشور لم يتم تعديله).
قم بتحديث صفحة المتصفح => يعود المحرر.
في الخطوة 2، إذا قمت بالنقر فوق زر “إلغاء” بدلاً من الضغط على مفتاح Esc، فإن السلوك يكون صحيحًا، أي أن المحرر لا يعود بعد تحديث المتصفح.
5 إعجابات
@jack2
لقد فعلتها للتو، نعم، تفعل بالضبط ما تقول. لست متأكدًا مما إذا كانت هذه مشكلة أم أنها من المفترض أن تفعل ذلك بالفعل.
3 إعجابات
j.jaffeux
(Joffrey Jaffeux)
29 مارس 2020، 8:46ص
4
لم أعد قادرًا على إعادة إنتاج هذه المشكلة، هل ما زلت قادرًا على إعادة إنتاجها في try.discourse.org من فضلك؟
jack2
29 مارس 2020، 9:32ص
5
نعم، لقد قمت بتكراره للتو على try.discourse.org .
انتظر! الأمر عشوائي! أحيانًا لا يحدث.
j.jaffeux
(Joffrey Jaffeux)
29 مارس 2020، 9:34ص
6
هل يمكنك العثور على تكرار متسق؟ لدي فكرة لإصلاح المشكلة، لكنني أفضل الحصول على تكرار متسق. من فضلك.
إعجاب واحد (1)
j.jaffeux
(Joffrey Jaffeux)
29 مارس 2020، 9:39ص
8
رائع! حسنًا، إذن فكرة الإصلاح الخاصة بي ستنجح على الأرجح، شكرًا جزيلًا لك
3 إعجابات
j.jaffeux
(Joffrey Jaffeux)
29 مارس 2020، 12:39م
9
كان هذا أكثر تعقيدًا بكثير مما توقعت، وأعتقد أن هذا سيصلح المشكلة:
master ← jjaffeux:draft-escape-fix
closed 06:41AM - 30 Mar 20 UTC
This fix will also make sure the bootbox will be correctly shown when using esca… pe.
Test cases:
- edit post, wait for draft check, escape => should totally hide composer
- edit post, add a char, escape => should show bootbox, and reduce or hide depending on choice
Same behavior is expected when pressing on cancel.
لقد طلبت مراجعات متعددة وقد يستغرق الأمر بعض الوقت للدمج، شكرًا مجددًا على وصف المشكلة المفصل
5 إعجابات
j.jaffeux
(Joffrey Jaffeux)
30 مارس 2020، 6:44ص
10
تحديث سريع، لقد أغلقت طلب الدمج (PR). كان الإصلاح يعمل، لكننا سنحاول معالجته على مستوى أعلى.
قد يستغرق بعض الوقت
4 إعجابات
sam
(Sam Saffron)
31 مارس 2020، 12:51ص
11
لقد قمت بإصلاح هذا:
committed 12:49AM - 31 Mar 20 UTC
When editing a post we were incorrectly saving a draft prior to user typing
Thi… s caused a bloat in the amount of drafts saved per user and inconsistency
around behavior of "escape" button.
It also lead to lots of warnings about draft conflicts when copying stuff
between posts.
The code is improved to use promises more appropriately, however further
changes are needed to clean up internals so methods consistently return
promises.
Too many methods in the controller sometimes return a promise and sometimes
an object. Long term the methods will become async and all of this will be
corrected.
هذا الملف معقد للغاية @eviltrout … لا يمكنني الانتظار حتى نضيف دعم async، لأن مجموعة كبيرة من الطرق تعيد أحيانًا وعدًا (promise) وأحيانًا أخرى تعيد كائنًا، مما يجعل استنتاج الأمور صعبًا للغاية.
في الوقت الحالي، أضفت بضع وعود وتجاوزات حتى لا نحفظ مسودة فورًا بعد النقر على زر التعديل. كان هذا أيضًا يُشعر @supermathie بالجنون لأنه جعل نسخ النص من عدة ردود إلى المنشور الأصلي معقدًا ومزعجًا للغاية.
7 إعجابات
هذا يفسّر الكثير من الإحباط الذي كنتُ أعانيه،
4 إعجابات
j.jaffeux
(Joffrey Jaffeux)
تم إغلاقه في
4 أبريل 2020، 2:57ص
13
تم إغلاق هذا الموضوع تلقائيًا بعد 4 أيام من آخر رد. لم يعد مسموحًا بإضافة ردود جديدة.