Ticket #63 (closed defect: fixed)

Opened 15 months ago

Last modified 13 months ago

Cancelling while opening a new document before it has started loading leaves user with no buffer

Reported by: jteh Owned by:
Priority: minor Milestone: 0.6
Component: Virtual buffers Version: trunk
Keywords: Cc:
Blocking: Blocked By:

Description

If a user opens a new document but cancels the request before the new document has actually started loading (i.e. the old document is still on screen), the user will be left with no virtual buffer.

Steps to Reproduce


  1. Load any page, as long as it's fully loaded and has a proper buffer.
  2. Go to an address that will take a while to load. The best way to do this is to use an address that will time out; e.g.  http://11.22.33.44/ . :) This causes the document to go busy.
  3. Now press escape.

Actual Results


After pressing escape, there is no longer an active virtual buffer. In technical terms, there's no way to get our buffer back because we killed it off when it went busy. Alt+tabbing fixes it. For some reason, simply tabbing does not.

Expected Results


The user should be returned to the buffer as it was prior to attempting to open the new document.

Additional Comments


I'm not sure how to solve this one. Even if we do get a state change back to read only and not busy (I would hope we do), there's no way to get it because we've killed our buffer object.

This all makes me wonder about the idea of not checking the busy state; i.e. the document gets loaded on focus and doc load complete, but busy isn't a requirement.

Change History

Changed 13 months ago by jteh

A solution might be to not kill the buffer if its document's state changes to busy. However, we would still not create a buffer if the document is busy.

Changed 13 months ago by mdcurran

Fixed in r2120. Could have some side affects. isAlive doesn't care about state_busy anymore.

Changed 13 months ago by jteh

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.