ce9a1c128c
* Show-page: split out some effects Break apart the large effect for easier tweaking, abiding by the 1-action-per-effect guideline. This also fixes the dependency list. After checking with Haffa, the exclusion of `pathname` and `hash` from the list is probably to avoid hitting `doResolveUri`. This should no longer be an issue after the split, so no suppression needed. * Show-page: fix "no content found" flicker ## Issue 1539: "Recently been seeing a flicker of "content not found" when direct loading content or channel links on mobile" ## Cause `doResolveUri` can only be called after the initial mount (in the effect), so at first render, `isResolvingUri` won't be set yet, so we briefly ended up in the "no content found" path. ## Change Factor out the logic that determines if we should send `doResolveUri`. This allows us to know that the upcoming effect will be sending `doResolveUri`, so we can hold off displaying "no content found". * Show-page: don't delay the spinner when resolving ## Reason The delay causes a blank page + footer, which looks bad. |
||
|---|---|---|
| .. | ||
| index.js | ||
| view.jsx | ||