Steps:
Actual result:
Expected results:
Browser: Firefox 76.0b8
(Also, it seems only the github login only works on the second attempt?)
Relevant code seems to be:
https://github.com/discourse/discourse/blob/84be92c0671b85ce9ba016b4e867784f7022e5f8/app/assets/javascripts/discourse/app/initializers/logout.js#L25
and
Reference to logout function could just be replaced with replaced with window.location.reload?
Behavior here isn’t great, refresh should, well, refresh, not “go home”. That said, I wonder if the implementation is correct, and the button is labeled wrong. If we don’t send people home, refreshing may result in 404 errors, if the page they were on was a PM, secure category, login_required site, etc.
That’s true, although in my case I was reading a bunch of public threads.
Also I guess it may be an issue that if the page was private, in that the that private content is still kinda visible behind the dialog before the button is clicked - so perhaps the page should refresh immediately?
This is certainly by design.
The client has no way of knowing if the topic you were looking is public or not, we could carry a lot of logic here to allow for this, but the way I see it being logged out is a super rare condition and in such a condition it should be thought as a “reset button”.
A “reset button” should take you back to home.
I guess I support changing the label here but I am not sure what we would call it “Refresh to Home Page” is just too long.
I guess change “Refresh” to “Home” then we can close this topic as complete @jomaxro?
The browser “refresh” button works well and doesn’t lose my place if it doesn’t have to. =)