[CLOSED] Missing intellisense on too big File

core-ai-bot commented 3 years ago

Issue by SeriousM Sunday Feb 23, 2014 at 23:02 GMT Originally opened as https://github.com/adobe/brackets/issues/6986

Version: sprint 36 experimental build 0.36.0-11506 (release 942505c3a) OS: Win7 x64, english Extensions: none

Hi, I found a strange bug and I want to share it with you.

I was playing around with the phaser.io framework v1.1.5 and included the not-minified version (phaser.js) of it but I noticed that the intellisense was missing. bracketserror

After changing to the minified version (phaser.min.js) and a reload of Brackets the intellisense was available. bracketsok

I was able to switch forth and back several times and the error was reproducible. Here is the gist to the test.js file.

var game = new Phaser.Game(800, 600, Phaser.AUTO, 'phaser-example', { preload: preload, create: create, update: update });

var s;

function create() {

    game.physics.setBoundsToWorld(true, true, true, false);

    s = game.add.tileSprite(0, 0, 800, 600, 'starfield');

    // auto-complete here:
    s.
}

I think the problem is about the size of the file. I know that the files pf phaser.io are JSLint'ed, therefor the minified version should not emit an error that causes this problem.

The problem is, that I can't produce such a big file to prove it. I tried to get a console output from Brackets but I failed.

I hope you can reproduce the error.

Best regards and keep up this awesome product!

core-ai-bot commented 3 years ago

Comment by dangoor Monday Mar 03, 2014 at 19:22 GMT

Medium to me to investigate as part of code hint refactoring and to make sure it's a known issue.

I believe there is a limit in our code hints code on file size that we use to ensure good performance of the editor. I'm going to double check with the info you've given us.

core-ai-bot commented 3 years ago

Comment by SeriousM Monday Mar 03, 2014 at 19:53 GMT

Alright, thank you. I think framework files like that should not be limited by size to avoid such issues.

core-ai-bot commented 3 years ago

Comment by dangoor Wednesday Aug 13, 2014 at 12:43 GMT

@SeriousM In the current release of Brackets (0.42), there is no longer a file size limit. However, there is a time limit, so if a file takes too long to process, we won't do hints for that file.

core-ai-bot commented 3 years ago

Comment by dangoor Wednesday Aug 13, 2014 at 12:43 GMT

Please take a look and let us know if it's working better for you!

core-ai-bot commented 3 years ago

Comment by dangoor Wednesday Aug 13, 2014 at 14:50 GMT

Correction: the size limit is now 10000 lines (up from 2000) so there is still a size limit but it should catch fewer files.

core-ai-bot commented 3 years ago

Comment by SeriousM Sunday Sep 21, 2014 at 06:23 GMT

Works, thanks! :+1:

brackets-archive / bracketsIssues

[CLOSED] Missing intellisense on too big File #6324