search

mikaeljorhult / brackets-todo

Brackets/Edge Code extension that displays all TODO comments in current document or project.
144 stars 37 forks source link

Doesn`t work #142

Closed BlueMystical closed 7 years ago

BlueMystical commented 7 years ago

Does not work on the latets Brackets 1.9 version. Not a single TODO comment, not inj JS, PHP or HMTL files.

BlueMystical commented 7 years ago

The problem seems to be with the "Comment Sintax" Regular Expresion. I had to tweak till i managed it to work:

Prefix: (?:// @)( Sufix: ):? (.?) ?(?=\/|\r\n|\n|$)

This works for both JS and PHP files, couldnt make it work with HTML files tough.

alelipona commented 7 years ago

What's file need to tweak?

BlueMystical commented 7 years ago

What's file need to tweak?

Open the Plugin settings, a window should pop, set the Prefix and Sufix fields as mentioned above.

alelipona commented 7 years ago

A window dosen't pop :(

BlueMystical commented 7 years ago

Press CTRL+ALT+T to open the TODO window. There is also an option in the 'View' menu. Press the Configuration Gear Button on the right. set the Prefix and Sufix fields as mentioned above.

sin titulo

alelipona commented 7 years ago

I have been pressing CTRL+ALT+T and TODO button to open the TODO window but the window is not opening. :-(

BlueMystical commented 7 years ago

do you have the latest version of both brackets and TODO? Try Re-installing the TODO plugin.

alelipona commented 7 years ago

do you have the latest version of both brackets and TODO? Yes screen shot 2017-07-12 at 07 45 11 screen shot 2017-07-12 at 07 52 17

Try Re-installing the TODO plugin I have try it but the TODO window is not opening. screen shot 2017-07-12 at 07 51 20

BlueMystical commented 7 years ago

Sorry, if nothing of before fixed the problem, i'm out of ideas.

alvint commented 7 years ago

Commit 018a020 is the culprit. The issue is that the backslashes are not escaped in modules/components/Defaults.js line 10. The line:

suffix: '):? *(.*?) ?(?=\/\*|-->|\*\/|\r\n|\n|\r|$)'

should be:

suffix: '):? *(.*?) ?(?=\\/\\*|-->|\\*\\/|\\r\\n|\\n|\\r|$)'

You should be able to edit your copy of this file (on macOS it's at /Users/yourname/Library/Application Support/Brackets/extensions/user/mikaeljolt.brackets-todo/modules/components/Defaults.js), remove any current "bracketsTodo" settings in your config, and restart Brackets to get things to work.

Can someone make a pull request?

BlueMystical commented 7 years ago

Works!, and under Windows the file is located in: %AppData%\Brackets\extensions\user\mikaeljorhult.brackets-todo\modules .

siphomateke commented 7 years ago

@alvint I created a pull request

therealcode commented 7 years ago

This worked wonder for me! The TODO Window never opened up.

On Windows, I did the change @alvint suggested but referencing @Jhollman for location in Windows. Good work guys!

Currently Using: Windows 10 64bit - i5 Brackets (Release 1.10 build 1.10.0-17483)

mikaeljorhult commented 7 years ago

A fix for this is in the just released version 0.9.5. Resetting the settings from the settings dialog uses the new defaults.