Шаги:
Фактический результат:
Ожидаемый результат:
Браузер: Firefox 76.0b8
(Кроме того, похоже, что вход через GitHub работает только со второй попытки?)
Соответствующий код, по-видимому, находится по ссылкам:
и
Можно ли просто заменить ссылку на функцию logout на window.location.reload?
Поведение здесь не очень хорошее: кнопка «Обновить» должна, собственно, обновлять страницу, а не «возвращать на главную». Тем не менее, интересно, правильно ли реализована эта функция, и, возможно, кнопка названа неверно. Если мы не будем перенаправлять пользователей на главную страницу, то при обновлении могут возникать ошибки 404, если страница, на которой они находились, была личной перепиской, защищённой категорией, сайтом с требованием входа и т.п.
Это верно, хотя в моём случае я читал множество публичных тем.
Также, возможно, проблема в том, что если страница приватная, то приватный контент всё ещё отчасти виден за диалоговым окном до нажатия кнопки — поэтому, возможно, страницу стоит обновлять сразу же?
Это, безусловно, сделано намеренно.
Клиент не может знать, является ли интересующая вас тема публичной или нет. Мы могли бы внедрить здесь много логики для решения этой задачи, но, по моему мнению, выход из системы — это крайне редкое состояние, и в таком случае его следует рассматривать как «кнопку сброса».
«Кнопка сброса» должна возвращать вас на главную страницу.
Я, пожалуй, поддерживаю изменение этой надписи, но не уверен, как её назвать: «Обновить до главной страницы» — слишком длинно.
Думаю, если изменить «Refresh» на «Home», то мы сможем закрыть эту тему как выполненную, @jomaxro?
Кнопка «Обновить» в браузере работает отлично и не сбрасывает моё место, если в этом нет необходимости. =)
PR отправлен:
