Page MenuHomePhabricator

Quicksand breaks the back button after a reload, et al
Open, NormalPublic

Description

  • Start on page A.
  • Perform a Quicksand navigation to page B.
  • Reload the page.
  • Press "Back".

Observed behavior:

  • You're on the pre-reload B with the wrong URL (/a/).

Expected behavior:

  • You're taken back to A with the right URL (/a/).

This behavior sort of makes sense internally and may be kind of hard to fix, but is definitely not what users expect or what happens without Quicksand, and shouldn't be what we do unless it's impossible to fix.


  • Introduce a syntax error into a Controller.
  • Navigate to that Controller via Quicksand.

Observed behavior:

  • No navigation occurs.

Expected behavior:

  • You get an actual indication that there was an error, and probably a hard navigation for most requests (e.g., if it's a GET at least?)

  • Visit any page with an anchor (/w/#quack).
  • Click anything.
  • Press back.

Observed behavior:

  • URL goes back; content does not.

Expected behavior:

  • The content should go back too.

  • With Quicksand:
  • Visit /dashboard/.
  • Click "Create Dashboard".
  • You get a dialog.
  • This dialog doesn't work (?) if you select "empty"?
  • Huh?

  • After T13094, the Quicksand behavior for things which redirect into file downloads is a little weird. For example, "Download Log" in Harbormaster creates a new history entry.

  • "Export as ICS" in Calendar doesn't work with Quicksand.

  • "Analyze Query Plans" in DarkConsole doesn't work terribly well with Quicksand. This button should probably always force a real navigation event.

  • Jumping to a particular line in the document renderer (Paste, Files, Diffusion) probably doesn't work with Quicksand if you're following a link there (D19350).

  • In Passphrase, clicking "Done" on the "Reveal Secret" dialog does not reload the page correctly if Quicksand is enabled.

Event Timeline

epriestley triaged this task as Normal priority.Feb 15 2018, 1:26 PM
epriestley created this task.
epriestley renamed this task from Quicksand breaks the back button after a reload to Quicksand breaks the back button after a reload, et al.Feb 15 2018, 1:31 PM
epriestley updated the task description. (Show Details)