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

Extremely slow performance opening jquery-1.6.min.js #706

Closed
peterflynn opened this issue Apr 20, 2012 · 6 comments
Closed

Extremely slow performance opening jquery-1.6.min.js #706

peterflynn opened this issue Apr 20, 2012 · 6 comments

Comments

@peterflynn
Copy link
Member

  1. Make sure you have the brackets-scenario repo
  2. Open the real_sites/PhoneGap_homepage folder
  3. Click on js/jquery-1.6.min.js

Result: opening the JS file hangs brackets for ~15 seconds.

There's a secondary bug that's probably separable: after the above steps, the UI is extremely sluggish. Clicking to move the cursor or opening a menu takes 1-2 seconds every time. This issue goes away if you disable JSLint (closing the JSLint panel) -- even though JSLint seems to have no effect on how slow it is to initially open the file.

@peterflynn
Copy link
Member Author

Unsurprisingly, this is not a regression -- still repros at least as far back as Sprint 5.

@gruehle
Copy link
Member

gruehle commented Apr 21, 2012

CodeMirror doesn't handle long lines well. See:

https://github.com/marijnh/CodeMirror2/issues/397#issuecomment-4180204

@peterflynn
Copy link
Member Author

Rats. Although that doesn't explain all the observed behavior -- e.g. I wonder why having the JSLint panel open in conjunction with that slows down not just CodeMirror operations, but also unrelated things like opening the menus.

@njx
Copy link
Contributor

njx commented Apr 24, 2012

Marking "to user story"--this should turn into a performance story on the backlog.

@pthiess
Copy link
Contributor

pthiess commented May 10, 2012

Moved to Trello, card 478

@pthiess pthiess closed this as completed May 10, 2012
@peterflynn
Copy link
Member Author

Looks like this may have been fixed by #1847

gideonthomas pushed a commit to gideonthomas/brackets that referenced this issue Apr 13, 2017
gideonthomas pushed a commit to gideonthomas/brackets that referenced this issue Apr 19, 2017
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

4 participants