Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

[Core][Live Preview]: Reloading function for Live Preview doesn't work correctly if you reload the page many times. #3761

Closed
julieyuan opened this issue May 9, 2013 · 6 comments
Labels
Milestone

Comments

@julieyuan
Copy link

Step:

  1. Open a html file in Brackets.
  2. Click the Live Preview Icon
  3. Click reload button on the Web page more than 10 times and you can see the Live Preview button's color changed from orange to gray.
    4.Click the Live preview icon again and there will pop up an error dialog.
  4. Click Ok button on the error dialog, then click Live preview icon again, you can open the web page succussfully.
  5. Close the previous working web page in the browser, and then observe Live Preview button, there will be a tooltip "Live Preview was cancelled because the page was closed in the browser" although the preview page is still opened in the browser.

Result:
​For step3, if reload the page in the browsers many times, the Live preview was cancelled. For step 4-6, the Live preview doesn't work correctly according user's behavior.

Expect:
​For step3, reloading the page shouldn't make the Live preview cancelled.
For step 6, it should not appear the error tooltip.

Env: Win7x64 ENU
build: Win 0.25.0-7294

Screenshot:
Step 4:
live preview-reload and error-step4- enu

Step 5:
live preview-reload and error-step5-enu

Step 6:
live preview-reload and error -step 6-enu

@ghost ghost assigned jasonsanjose May 9, 2013
@njx
Copy link

njx commented May 13, 2013

Reviewed. Opening for sprint 25, medium priority, as this was easily reproducible and I could imagine people running into it. The issue could be either in the static server code or in the connection to the browser. @jasonsanjose - feel free to reassign if necessary.

@pthiess
Copy link
Contributor

pthiess commented May 13, 2013

Reviewed - Medium priority

@jasonsanjose
Copy link
Member

Inspector is reporting Inspected frame has gone when refreshing the browser quickly.

@jasonsanjose
Copy link
Member

Oops. Accidentally autoclosed. FBNC @julieyuan.

@jasonsanjose jasonsanjose reopened this May 21, 2013
@ghost ghost assigned jasonsanjose and Zhuyun May 21, 2013
@jasonsanjose
Copy link
Member

Hmm, I can't seem to assign to @julieyuan. @Zhuyun can you triage or reassign?

@julieyuan
Copy link
Author

Regressed this bug in Build 0.25.0-7677. It is fixed.
Please refer to snapshot:
bug 3761 fixed in 0 25 0 7699

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants