Docs for Prefs code hints

[abose]

• Update wiki pages on how to define code hints for preferences
• JS Docs for define preference.
• remove data.json
• Find out core preferences that has not used the define preference API and add prefs docs.
• Update preference WIKI pages.
• Mail brackets committers/slack channel about the new API. Create entry in release notes for 1.4 for API change.

[abose]

Update the wikis
https://github.com/adobe/brackets/wiki/Preferences-System and https://github.com/adobe/brackets/wiki/How-to-Use-Brackets#preferences

[sprintr]

Find out core preferences that has not used the define preference API and add prefs docs.

I think @marcelgerber covered this story in #11197.

[abose]

Yes, but I see that some properties are still left not defined as a preference. But that could go in at a later time.

[marcelgerber]

@abose Which preferences do you still see un-defined?
From what I can tell, every preference is defined, unless you're talking about things that should be preferences but aren't (JS Code Hints comes to mind)

[abose]

@nethip I remember you mentioned about undefined preferences, Could you specify which ones did you see as udefined?

[sprintr]

I don't think there are any undefined preferences, since we suppress defaultExtension pref, so it got excluded from code hints. _windowsScrollFix was excluded intentionally.

[nethip]

@sprintr I am working on this story #11056 wherein we are going to display default preferences along with brackets.json side by side in the split view. Right now I am regenerating default preferences by looping through all preferences using this call PreferencesManager.getAllPreferences(). While iterating through all preferences, I found that the following preferences do not have description field.

"linting.asyncTimeout": 10000,
"linting.usePreferredOnly": false,
"codehint.TagHints": true,
"codehint.AttrHints": true,
"codehint.SpecialCharHints": true,
"codehint.CssPropHints": true,
"staticserver.port": 0,
"codehint.UrlCodeHints": true,

[marcelgerber]

Those are the exact ones we altered in #11197.

[nethip]

@marcelgerber Yes I see that these are now going through definePreference, but the description is missing for all the properties.

[marcelgerber]

I see, but #11201 should have fixed that.

[nethip]

@marcelgerber you are right. I must be working on my branch which might have not this change.Thanks for pointing out the PR.

[sprintr]

I believe this can now be closed since all the checklist items are done.

[abose]

Thanks @sprintr . Closing as done.