- Click a topic from topic list
- You see a nice spinner
- Click back
- Click topic again
- No spinner shows up
Yeah, this is what I was thinking of when I said regression. Could be the CSS way isn’t working in this case. We might want to backport this fix, whatever it is, @eviltrout
I don’t think it was a regression – basically we didn’t handle the loading case when a route was entered that returned a promise instead of a dummy object that wasn’t loaded yet.
I’ve fixed it by adding a generic loading template. A side effect of this is that more routes might show the loading spinner than did previously, if they return promises. We can tweak this if we don’t like it.
I’d prefer that, especially if they are performing an entire page load. I’ve seen several scenarios previously that show no indication, so maybe this will address that.
Granted, I haven’t been annoyed with it enough to report it…