Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Dead tab when loading large content #8045

Closed
bsclifton opened this issue Apr 2, 2017 · 6 comments
Closed

Dead tab when loading large content #8045

bsclifton opened this issue Apr 2, 2017 · 6 comments

Comments

@bsclifton
Copy link
Member

Browser UI elements (home button, reload button) are blocked and unresponsive when page is loading a large amount of content

I know this has happened to @BrendanEich recently. Repro steps by @NejcZdovc:

  1. Go to https://github.com/brave/browser-laptop/pull/7795/files
  2. npm-shrinkwrap.json is not expanded by default
  3. Click Load diff and wait a few seconds
  4. Click the reload button (next to the URL bar) a few times
  5. Notice nothing happens
@bsclifton bsclifton added this to the 0.14.3 milestone Apr 2, 2017
@luixxiul
Copy link
Contributor

luixxiul commented Apr 2, 2017

I have noticed it as well when I loaded large pictures on a single page on slow Internet connection. I thought back/forward/reload button was not available until onload event was fired.

@bsclifton
Copy link
Member Author

bsclifton commented Apr 2, 2017

@luixxiul great find! I believe this can be the problem. We are likely having cases where the onload never fires because of a network error, etc. We should look at changing this behavior or possibly adding a timeout

cc: @bridiver

@bsclifton bsclifton added needs-owner ♞ This issue is tagged for an upcoming release but has no owner. and removed needs-owner ♞ This issue is tagged for an upcoming release but has no owner. labels Apr 4, 2017
@alexwykoff alexwykoff modified the milestones: 1.0.0, 0.15.1 Apr 18, 2017
@bsclifton bsclifton removed the needs-owner ♞ This issue is tagged for an upcoming release but has no owner. label Jun 22, 2017
@bsclifton
Copy link
Member Author

This appears to be fixed 😄

@luixxiul @srirambv @LaurenWags could you try the steps listed here and confirm it's fixed? (to help make sure I'm not crazy?). Thanks 😄

@LaurenWags
Copy link
Member

@bsclifton - looked ok to me (MacOS)

@srirambv
Copy link
Collaborator

@bsclifton Looks good on Windows 10 x64, although the onclick response is a little delayed and not instant as for normal page content

@bsclifton
Copy link
Member Author

awesome! I'll mark this as closed. I think the slowness is captured pretty well with our perf tracking issue

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

No branches or pull requests

5 participants