الخطوات:
النتيجة الفعلية:
النتيجة المتوقعة:
المتصفح: Firefox 76.0b8
(أيضًا، يبدو أن تسجيل الدخول عبر GitHub يعمل فقط في المحاولة الثانية؟)
يبدو أن الكود ذو الصلة هو:
و
هل يمكن ببساطة استبدال الإشارة إلى دالة logout بـ window.location.reload؟
السلوك هنا ليس جيدًا، حيث يجب أن يؤدي التحديث إلى إعادة تحميل الصفحة، وليس “الذهاب إلى الصفحة الرئيسية”. مع ذلك، أتساءل عما إذا كان التنفيذ صحيحًا وأن الزر مُسمّى بشكل خاطئ. إذا لم نوجه المستخدمين إلى الصفحة الرئيسية، فقد يؤدي التحديث إلى ظهور أخطاء 404، خاصة إذا كانت الصفحة التي كانوا عليها خاصة برسالة مباشرة، أو فئة آمنة، أو موقعًا يتطلب تسجيل الدخول، وما إلى ذلك.
هذا صحيح، على الرغم من أنني في حالتي كنت أقرأ عددًا من المواضيع العامة.
كما أعتقد أن هناك مشكلة محتملة، وهي أنه إذا كانت الصفحة خاصة، فإن المحتوى الخاص يظل مرئيًا بشكل ما خلف النافذة المنبثقة قبل النقر على الزر، لذا ربما يجب أن يتم تحديث الصفحة فورًا؟
هذا بالتأكيد عن قصد.
لا يمتلك العميل أي طريقة لمعرفة ما إذا كان الموضوع الذي تبحث عنه عامًا أم لا. يمكننا حمل الكثير من المنطق هنا للسماح بذلك، لكن الطريقة التي أراها في تسجيل الخروج هي حالة نادرة جدًا، وفي مثل هذه الحالة يجب اعتبارها “زر إعادة تعيين”.
يجب أن يأخذك “زر إعادة التعيين” إلى الصفحة الرئيسية.
أعتقد أنني أؤيد تغيير التسمية هنا، لكنني لست متأكدًا مما يجب أن نسميه. “تحديث إلى الصفحة الرئيسية” طويل جدًا.
أظن أنه إذا قمنا بتغيير “تحديث” إلى “الرئيسية”، فيمكننا إغلاق هذا الموضوع باعتباره مكتملاً @jomaxro؟
زر “تحديث” المتصفح يعمل بشكل جيد ولا يفقد موقعي إذا لم يكن ذلك ضروريًا =)
تم رفع طلب السحب (PR):
