From fc5d8e8cbfc3d8e02d227b2e8e26c5fdb181d4c4 Mon Sep 17 00:00:00 2001 From: Dan Date: Fri, 26 Aug 2016 21:27:23 -0400 Subject: [PATCH 01/11] Adds autofill detail tokens to the autocomplete attribute --- .../default/HTMLCodeHints/HtmlAttributes.json | 62 +++++++++++-------- 1 file changed, 35 insertions(+), 27 deletions(-) diff --git a/src/extensions/default/HTMLCodeHints/HtmlAttributes.json b/src/extensions/default/HTMLCodeHints/HtmlAttributes.json index d7d24e12b78..4f63e8c0f55 100644 --- a/src/extensions/default/HTMLCodeHints/HtmlAttributes.json +++ b/src/extensions/default/HTMLCodeHints/HtmlAttributes.json @@ -8,29 +8,29 @@ "dropzone": { "attribOption": ["copy", "move", "link"], "global": "true" }, "hidden": { "attribOption": ["hidden"], "global": "true" }, "id": { "attribOption": [], "global": "true", "type": "cssId" }, - "lang": { "attribOption": ["ab", "aa", "af", "sq", "am", "ar", "an", "hy", "as", "ay", "az", "ba", "eu", "bn", "dz", "bh", "bi", "br", - "bg", "my", "be", "km", "ca", "zh", "co", "hr", "cs", "da", "nl", "en", "eo", "et", "fo", "fa", "fi", "fr", - "fy", "gl", "gd", "gv", "ka", "de", "el", "kl", "gn", "gu", "ht", "ha", "he", "hi", "hu", "is", "io", "id", - "ia", "ie", "iu", "ik", "ga", "it", "ja", "jv", "kn", "ks", "kk", "rw", "ky", "rn", "ko", "ku", "lo", "la", - "lv", "li", "ln", "lt", "mk", "mg", "ms", "ml", "mt", "mi", "mr", "mo", "mn", "na", "ne", "no", "oc", "or", - "om", "ps", "pl", "pt", "pa", "qu", "rm", "ro", "ru", "sz", "sm", "sg", "sa", "sr", "sh", "st", "tn", "sn", - "ii", "sd", "si", "ss", "sk", "sl", "so", "es", "su", "sw", "sv", "tl", "tg", "ta", "tt", "te", "th", "bo", - "ti", "to", "ts", "tr", "tk", "tw", "ug", "uk", "ur", "uz", "vi", "vo", "wa", "cy", "wo", "xh", "yi", "yo", - "zu"], + "lang": { "attribOption": ["ab", "aa", "af", "sq", "am", "ar", "an", "hy", "as", "ay", "az", "ba", "eu", "bn", "dz", "bh", "bi", "br", + "bg", "my", "be", "km", "ca", "zh", "co", "hr", "cs", "da", "nl", "en", "eo", "et", "fo", "fa", "fi", "fr", + "fy", "gl", "gd", "gv", "ka", "de", "el", "kl", "gn", "gu", "ht", "ha", "he", "hi", "hu", "is", "io", "id", + "ia", "ie", "iu", "ik", "ga", "it", "ja", "jv", "kn", "ks", "kk", "rw", "ky", "rn", "ko", "ku", "lo", "la", + "lv", "li", "ln", "lt", "mk", "mg", "ms", "ml", "mt", "mi", "mr", "mo", "mn", "na", "ne", "no", "oc", "or", + "om", "ps", "pl", "pt", "pa", "qu", "rm", "ro", "ru", "sz", "sm", "sg", "sa", "sr", "sh", "st", "tn", "sn", + "ii", "sd", "si", "ss", "sk", "sl", "so", "es", "su", "sw", "sv", "tl", "tg", "ta", "tt", "te", "th", "bo", + "ti", "to", "ts", "tr", "tk", "tw", "ug", "uk", "ur", "uz", "vi", "vo", "wa", "cy", "wo", "xh", "yi", "yo", + "zu"], "global": "true" }, - "role": { "attribOption": ["alert", "alertdialog", "article", "application", "banner", "button", "checkbox", "columnheader", "combobox", - "complementary", "contentinfo", "definition", "directory", "dialog", "document", "form", "grid", "gridcell", - "group", "heading", "img", "link", "list", "listbox", "listitem", "log", "main", "marquee", "math", "menu", - "menubar", "menuitem", "menuitemcheckbox", "menuitemradio", "navigation", "note", "option", "presentation", - "progressbar", "radio", "radiogroup", "region", "row", "rowgroup", "rowheader", "scrollbar", "search", - "separator", "slider", "spinbutton", "status", "tab", "tablist", "tabpanel", "textbox", "timer", "toolbar", - "tooltip", "tree", "treegrid", "treeitem"], + "role": { "attribOption": ["alert", "alertdialog", "article", "application", "banner", "button", "checkbox", "columnheader", "combobox", + "complementary", "contentinfo", "definition", "directory", "dialog", "document", "form", "grid", "gridcell", + "group", "heading", "img", "link", "list", "listbox", "listitem", "log", "main", "marquee", "math", "menu", + "menubar", "menuitem", "menuitemcheckbox", "menuitemradio", "navigation", "note", "option", "presentation", + "progressbar", "radio", "radiogroup", "region", "row", "rowgroup", "rowheader", "scrollbar", "search", + "separator", "slider", "spinbutton", "status", "tab", "tablist", "tabpanel", "textbox", "timer", "toolbar", + "tooltip", "tree", "treegrid", "treeitem"], "global": "true" }, "spellcheck": { "attribOption": [], "global": "true", "type": "boolean" }, "style": { "attribOption": [], "global": "true", "type": "style" }, "tabindex": { "attribOption": [], "global": "true" }, "title": { "attribOption": [], "global": "true" }, - + "onabort": { "attribOption": [], "global": "true" }, "onblur": { "attribOption": [], "global": "true" }, "oncanplay": { "attribOption": [], "global": "true" }, @@ -85,7 +85,7 @@ "ontimeupdate": { "attribOption": [], "global": "true" }, "onvolumechange": { "attribOption": [], "global": "true" }, "onwaiting": { "attribOption": [], "global": "true" }, - + "accept": { "attribOption": ["text/html", "text/plain", "application/msword", "application/msexcel", "application/postscript", "application/x-zip-compressed", "application/pdf", "application/rtf", "video/x-msvideo", "video/quicktime", "video/x-mpeg2", "audio/x-pn/realaudio", "audio/x-mpeg", "audio/x-waw", "audio/x-aiff", "audio/basic", @@ -97,18 +97,26 @@ "alt": { "attribOption": [] }, "archive": { "attribOption": [] }, "async": { "attribOption": [], "type": "flag" }, - "autocomplete": { "attribOption": ["off", "on"] }, + "autocomplete": { "attribOption": ["additional-name", "address-level1", "address-level2", "address-level3", "address-level4", "address-line1", + "address-line2", "address-line3", "bday", "bday-year", "bday-day", "bday-month", "billing", + "cc-additional-name", "cc-csc", "cc-exp", "cc-exp-month", "cc-exp-year", "cc-family-name", "cc-given-name", + "cc-name", "cc-number", "cc-type", "country", "country-name", "current-password", "email", "family-name", + "fax", "given-name", "home", "honorific-prefix", "honorific-suffix", "impp", "language", "mobile", "name", + "new-password", "nickname", "organization", "organization-title", "pager", "photo", "postal-code", "sex", + "shipping", "street-address", "tel-area-code", "tel", "tel-country-code", "tel-extension", + "tel-local", "tel-local-prefix", "tel-local-suffix", "tel-national", "transaction-amount", + "transaction-currency", "url", "username", "work"] }, "autofocus": { "attribOption": [], "type": "flag" }, "autoplay": { "attribOption": [], "type": "flag" }, "behavior": { "attribOption": ["scroll", "slide", "alternate"] }, "bgcolor": { "attribOption": [], "type": "color" }, "border": { "attribOption": [] }, "challenge": { "attribOption": [] }, - "charset": { "attribOption": ["iso-8859-1", "utf-8", "shift_jis", "euc-jp", "big5", "gb2312", "euc-kr", "din_66003-kr", "ns_4551-1-kr", - "sen_850200_b", "csISO2022jp", "hz-gb-2312", "ibm852", "ibm866", "irv", "iso-2022-kr", "iso-8859-2", - "iso-8859-3", "iso-8859-4", "iso-8859-5", "iso-8859-6", "iso-8859-7", "iso-8859-8", "iso-8859-9", "koi8-r", - "ks_c_5601", "windows-1250", "windows-1251", "windows-1252", "windows-1253", "windows-1254", "windows-1255", - "windows-1256", "windows-1257", "windows-1258", "windows-874", "x-euc", "asmo-708", "dos-720", "dos-862", + "charset": { "attribOption": ["iso-8859-1", "utf-8", "shift_jis", "euc-jp", "big5", "gb2312", "euc-kr", "din_66003-kr", "ns_4551-1-kr", + "sen_850200_b", "csISO2022jp", "hz-gb-2312", "ibm852", "ibm866", "irv", "iso-2022-kr", "iso-8859-2", + "iso-8859-3", "iso-8859-4", "iso-8859-5", "iso-8859-6", "iso-8859-7", "iso-8859-8", "iso-8859-9", "koi8-r", + "ks_c_5601", "windows-1250", "windows-1251", "windows-1252", "windows-1253", "windows-1254", "windows-1255", + "windows-1256", "windows-1257", "windows-1258", "windows-874", "x-euc", "asmo-708", "dos-720", "dos-862", "dos-874", "cp866", "cp1256"] }, "checked": { "attribOption": [], "type": "flag" }, "cite": { "attribOption": [] }, @@ -177,9 +185,9 @@ "preload": { "attribOption": ["auto", "metadata", "none"] }, "pubdate": { "attribOption": [] }, "radiogroup": { "attribOption": [] }, - "rel": { "attribOption": ["alternate", "author", "bookmark", "help", "license", "next", "nofollow", "noreferrer", "prefetch", + "rel": { "attribOption": ["alternate", "author", "bookmark", "help", "license", "next", "nofollow", "noreferrer", "prefetch", "prev", "search", "sidebar", "tag", "external"] }, - "link/rel": { "attribOption": ["alternate", "author", "help", "icon", "license", "next", "pingback", "prefetch", "prev", "search", + "link/rel": { "attribOption": ["alternate", "author", "help", "icon", "license", "next", "pingback", "prefetch", "prev", "search", "sidebar", "stylesheet", "tag"] }, "readonly": { "attribOption": [], "type": "flag" }, "required": { "attribOption": [], "type": "flag" }, @@ -211,7 +219,7 @@ "link/type": { "attribOption": ["text/css"] }, "menu/type": { "attribOption": ["context", "list", "toolbar"] }, "ol/type": { "attribOption": ["1", "a", "A", "i", "I"] }, - "script/type": { "attribOption": ["text/javascript", "text/ecmascript", "text/jscript", "text/livescript", "text/tcl", "text/x-javascript", "text/x-ecmascript", + "script/type": { "attribOption": ["text/javascript", "text/ecmascript", "text/jscript", "text/livescript", "text/tcl", "text/x-javascript", "text/x-ecmascript", "application/x-javascript", "application/x-ecmascript", "application/javascript", "application/ecmascript", "text/babel", "text/jsx"] }, "style/type": { "attribOption": ["text/css"] }, From 1ad12c40c6a912ebc755a32bedd9eb993f3662cf Mon Sep 17 00:00:00 2001 From: ficristo Date: Mon, 29 Aug 2016 20:27:50 +0200 Subject: [PATCH 02/11] Add an ISSUE_TEMPLATE --- ISSUE_TEMPLATE.md | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 ISSUE_TEMPLATE.md diff --git a/ISSUE_TEMPLATE.md b/ISSUE_TEMPLATE.md new file mode 100644 index 00000000000..74a662687cf --- /dev/null +++ b/ISSUE_TEMPLATE.md @@ -0,0 +1,27 @@ +### Prerequisites + +* [ ] Can you reproduce the problem with `Debug \ Reload Without Extensions`? +* [ ] Did you perform a cursory search to see if your bug or enhancement is already reported? +* [ ] Did you read the [Troubleshooting guide](https://github.com/adobe/brackets/wiki/Troubleshooting)? + +For more information on how to write a good bug report read [here](https://github.com/adobe/brackets/wiki/How-to-Report-an-Issue) +For more information on how to contribute read [here](https://github.com/adobe/brackets/blob/master/CONTRIBUTING.md) + +### Description + +[Description of the bug or feature] + +### Steps to Reproduce + +1. [First Step] +2. [Second Step] +3. [and so on...] + +**Expected behavior:** [What you expected to happen] + +**Actual behavior:** [What actually happened] + +### Versions + +Please include the OS and what version of the OS you're running. +Please include the version of Brackets. You can find it under `Help \ About Brackets` From fc81b303a58acbe0a4e6d712a78581f3952ace04 Mon Sep 17 00:00:00 2001 From: Nathan J Plummer Date: Tue, 6 Sep 2016 22:53:56 -0400 Subject: [PATCH 03/11] Add link to Linux Guide for frequent problems overreported in GitHub issues. --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index 3bc231e10d4..dda66500c0b 100644 --- a/README.md +++ b/README.md @@ -57,6 +57,8 @@ Having problems starting Brackets the first time, or not sure how to use Bracket review [Troubleshooting](https://github.com/adobe/brackets/wiki/Troubleshooting), which helps you to fix common problems and find extra help if needed. +For a list of common Linux issues and workarounds you can [visit this guide](https://nathanjplummer.github.io/Brackets/). + Helping Brackets ---------------- From dedd539e60dee8f54da87bd722fe71ec681ec14a Mon Sep 17 00:00:00 2001 From: Dayton Allen Date: Fri, 9 Sep 2016 12:20:38 -0500 Subject: [PATCH 04/11] Fixed leak in menujs file --- src/command/Menus.js | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/src/command/Menus.js b/src/command/Menus.js index 71cf3b54808..d002a04d09a 100644 --- a/src/command/Menus.js +++ b/src/command/Menus.js @@ -178,6 +178,19 @@ define(function (require, exports, module) { return contextMenuMap[id]; } + /** + * Removes the attached event listeners from the corresponding object. + * @param {Command} command + */ + function removeMenuItemEventListeners(command) { + command + .off("enabledStateChange", command._enabledChanged) + .off("checkedStateChange", command._checkedChanged) + .off("nameChange", command._nameChanged) + .off("keyBindingAdded", command._keyBindingAdded) + .off("keyBindingRemoved", command._keyBindingRemoved); + } + /** * Check whether a ContextMenu exists for the given id. * @param {string} id @@ -444,9 +457,10 @@ define(function (require, exports, module) { console.error("removeMenuItem(): command not found: " + command); return; } - + removeMenuItemEventListeners(commandObj); commandID = command; } else { + removeMenuItemEventListeners(command); commandID = command.getID(); } menuItemID = this._getMenuItemId(commandID); From 970d69a8b4b7f04cbc97fa3280ac9f90266e253e Mon Sep 17 00:00:00 2001 From: Dayton Allen Date: Fri, 9 Sep 2016 21:19:49 -0500 Subject: [PATCH 05/11] Added test case for detaching menu items' event listeners --- test/spec/Menu-test.js | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) diff --git a/test/spec/Menu-test.js b/test/spec/Menu-test.js index a1337b289dd..1bb184a52b9 100644 --- a/test/spec/Menu-test.js +++ b/test/spec/Menu-test.js @@ -734,6 +734,33 @@ define(function (require, exports, module) { expect(menu).toBeTruthy(); // Verify that we got this far... }); }); + + it("should add then remove new menu item ensuring event listeners have also been detached", function () { + runs(function () { + var commandId = "Menu-test.removeMenuItem.command4"; + var menuItemId = "menu-test-removeMenuItem4"; + CommandManager.register("Brackets Test Command Custom", commandId, function () {}); + var menu = Menus.addMenu("Custom", menuItemId); + var $listItems = testWindow.$("#menu-custom > ul").children(); + expect($listItems.length).toBe(0); + + // Re-use commands that are already registered + var menuItem = menu.addMenuItem(commandId); + expect(menuItem).toBeTruthy(); + expect(menuItem).toBeDefined(); + + $listItems = menuDOMChildren(menuItemId); + expect($listItems.length).toBe(1); + + var command = CommandManager.get(commandId); + expect(typeof (command)).toBe("object"); + + command.on("enabledStateChange", command._enabledChanged); + var length = command._eventHandlers.enabledStateChange.length; + menu.removeMenuItem(command); + expect(command._eventHandlers.enabledStateChange.length).toBeLessThan(length); + }); + }); }); From 3b409484cb14986a3b87b462746834c8a2597ca0 Mon Sep 17 00:00:00 2001 From: Dayton Allen Date: Sat, 10 Sep 2016 00:09:58 -0500 Subject: [PATCH 06/11] Fixed removeMenuItemListeners test case. --- test/spec/Menu-test.js | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/test/spec/Menu-test.js b/test/spec/Menu-test.js index 1bb184a52b9..74317a79843 100644 --- a/test/spec/Menu-test.js +++ b/test/spec/Menu-test.js @@ -746,19 +746,25 @@ define(function (require, exports, module) { // Re-use commands that are already registered var menuItem = menu.addMenuItem(commandId); + expect(menuItem).toBeTruthy(); expect(menuItem).toBeDefined(); $listItems = menuDOMChildren(menuItemId); expect($listItems.length).toBe(1); - + var command = CommandManager.get(commandId); expect(typeof (command)).toBe("object"); - - command.on("enabledStateChange", command._enabledChanged); - var length = command._eventHandlers.enabledStateChange.length; + + // Test event + command.on("nameChange", menuItem._nameChanged); + + expect(typeof (command._eventHandlers.nameChange)).toBe("object"); + menu.removeMenuItem(command); - expect(command._eventHandlers.enabledStateChange.length).toBeLessThan(length); + + // Check if test event has been removed + expect(command._eventHandlers.nameChange).toBeUndefined(); }); }); }); From 26bb4c9139b70b469cbb5d63742fdf9336a8e152 Mon Sep 17 00:00:00 2001 From: Dayton Allen Date: Sat, 10 Sep 2016 08:52:16 -0500 Subject: [PATCH 07/11] Cleaned up removeMenuItemListeners test case. --- test/spec/Menu-test.js | 26 +++++++++++--------------- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/test/spec/Menu-test.js b/test/spec/Menu-test.js index 74317a79843..1befd94101f 100644 --- a/test/spec/Menu-test.js +++ b/test/spec/Menu-test.js @@ -737,34 +737,30 @@ define(function (require, exports, module) { it("should add then remove new menu item ensuring event listeners have also been detached", function () { runs(function () { - var commandId = "Menu-test.removeMenuItem.command4"; var menuItemId = "menu-test-removeMenuItem4"; + var commandId = "Menu-test.removeMenuItem.command4"; CommandManager.register("Brackets Test Command Custom", commandId, function () {}); var menu = Menus.addMenu("Custom", menuItemId); - var $listItems = testWindow.$("#menu-custom > ul").children(); - expect($listItems.length).toBe(0); - - // Re-use commands that are already registered - var menuItem = menu.addMenuItem(commandId); - expect(menuItem).toBeTruthy(); - expect(menuItem).toBeDefined(); - $listItems = menuDOMChildren(menuItemId); - expect($listItems.length).toBe(1); + var menuItem = menu.addMenuItem(commandId); var command = CommandManager.get(commandId); - expect(typeof (command)).toBe("object"); - - // Test event - command.on("nameChange", menuItem._nameChanged); + expect(typeof (command._eventHandlers.enabledStateChange)).toBe("object"); + expect(typeof (command._eventHandlers.checkedStateChange)).toBe("object"); expect(typeof (command._eventHandlers.nameChange)).toBe("object"); + expect(typeof (command._eventHandlers.keyBindingAdded)).toBe("object"); + expect(typeof (command._eventHandlers.keyBindingRemoved)).toBe("object"); menu.removeMenuItem(command); - // Check if test event has been removed + // Check if attached events have been removed expect(command._eventHandlers.nameChange).toBeUndefined(); + expect(command._eventHandlers.checkedStateChange).toBeUndefined(); + expect(command._eventHandlers.KeyBindingAdded).toBeUndefined(); + expect(command._eventHandlers.keyBindingRemoved).toBeUndefined(); + expect(command._eventHandlers.enabledStateChange).toBeUndefined(); }); }); }); From 7bce131ba8cfdb2d133b1ca1d72a0f273c2c0944 Mon Sep 17 00:00:00 2001 From: Dayton J Allen Date: Sat, 10 Sep 2016 14:44:44 -0500 Subject: [PATCH 08/11] Applied suggested fixes. --- src/command/Menus.js | 19 ++++++++++--------- test/spec/Menu-test.js | 6 +++++- 2 files changed, 15 insertions(+), 10 deletions(-) diff --git a/src/command/Menus.js b/src/command/Menus.js index d002a04d09a..72bd849d4ee 100644 --- a/src/command/Menus.js +++ b/src/command/Menus.js @@ -182,13 +182,13 @@ define(function (require, exports, module) { * Removes the attached event listeners from the corresponding object. * @param {Command} command */ - function removeMenuItemEventListeners(command) { - command - .off("enabledStateChange", command._enabledChanged) - .off("checkedStateChange", command._checkedChanged) - .off("nameChange", command._nameChanged) - .off("keyBindingAdded", command._keyBindingAdded) - .off("keyBindingRemoved", command._keyBindingRemoved); + function removeMenuItemEventListeners(menuItem) { + menuItem._command + .off("enabledStateChange", menuItem._enabledChanged) + .off("checkedStateChange", menuItem._checkedChanged) + .off("nameChange", menuItem._nameChanged) + .off("keyBindingAdded", menuItem._keyBindingAdded) + .off("keyBindingRemoved", menuItem._keyBindingRemoved); } /** @@ -457,14 +457,15 @@ define(function (require, exports, module) { console.error("removeMenuItem(): command not found: " + command); return; } - removeMenuItemEventListeners(commandObj); commandID = command; } else { - removeMenuItemEventListeners(command); commandID = command.getID(); } menuItemID = this._getMenuItemId(commandID); + var menuItem = getMenuItem(menuItemID); + removeMenuItemEventListeners(menuItem); + if (_isHTMLMenu(this.id)) { // Targeting parent to get the menu item and the
  • that contains it $(_getHTMLMenuItem(menuItemID)).parent().remove(); diff --git a/test/spec/Menu-test.js b/test/spec/Menu-test.js index 1befd94101f..ebf8e5639f3 100644 --- a/test/spec/Menu-test.js +++ b/test/spec/Menu-test.js @@ -746,21 +746,25 @@ define(function (require, exports, module) { var menuItem = menu.addMenuItem(commandId); var command = CommandManager.get(commandId); + command.on("testEvent", function () {}); expect(typeof (command._eventHandlers.enabledStateChange)).toBe("object"); expect(typeof (command._eventHandlers.checkedStateChange)).toBe("object"); expect(typeof (command._eventHandlers.nameChange)).toBe("object"); expect(typeof (command._eventHandlers.keyBindingAdded)).toBe("object"); expect(typeof (command._eventHandlers.keyBindingRemoved)).toBe("object"); + expect(typeof (command._eventHandlers.testEvent)).toBe("object"); menu.removeMenuItem(command); // Check if attached events have been removed expect(command._eventHandlers.nameChange).toBeUndefined(); expect(command._eventHandlers.checkedStateChange).toBeUndefined(); - expect(command._eventHandlers.KeyBindingAdded).toBeUndefined(); + expect(command._eventHandlers.keyBindingAdded).toBeUndefined(); expect(command._eventHandlers.keyBindingRemoved).toBeUndefined(); expect(command._eventHandlers.enabledStateChange).toBeUndefined(); + expect(typeof (command._eventHandlers.testEvent)).toBe("object"); + }); }); }); From d89738d3c2ae3511511e4ca9575122158a1281b0 Mon Sep 17 00:00:00 2001 From: Martin Zagora Date: Wed, 14 Sep 2016 20:16:47 +1000 Subject: [PATCH 09/11] set eslints max-len to 120 --- .eslintrc.json | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.eslintrc.json b/.eslintrc.json index 4e9117a0459..16b5f74189a 100644 --- a/.eslintrc.json +++ b/.eslintrc.json @@ -46,7 +46,8 @@ "valid-typeof": 2, "no-trailing-spaces": 0, - "eol-last": 0 + "eol-last": 0, + "max-len": [1, 120] }, "globals": { "brackets": false, From 2286cd1db260fcfb130435161416a55b007472a9 Mon Sep 17 00:00:00 2001 From: Dan Date: Wed, 14 Sep 2016 15:30:41 -0400 Subject: [PATCH 10/11] Adds `on` `off` values --- src/extensions/default/HTMLCodeHints/HtmlAttributes.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/extensions/default/HTMLCodeHints/HtmlAttributes.json b/src/extensions/default/HTMLCodeHints/HtmlAttributes.json index 4f63e8c0f55..2762ebff2ba 100644 --- a/src/extensions/default/HTMLCodeHints/HtmlAttributes.json +++ b/src/extensions/default/HTMLCodeHints/HtmlAttributes.json @@ -102,7 +102,7 @@ "cc-additional-name", "cc-csc", "cc-exp", "cc-exp-month", "cc-exp-year", "cc-family-name", "cc-given-name", "cc-name", "cc-number", "cc-type", "country", "country-name", "current-password", "email", "family-name", "fax", "given-name", "home", "honorific-prefix", "honorific-suffix", "impp", "language", "mobile", "name", - "new-password", "nickname", "organization", "organization-title", "pager", "photo", "postal-code", "sex", + "new-password", "nickname", "off", "on" "organization", "organization-title", "pager", "photo", "postal-code", "sex", "shipping", "street-address", "tel-area-code", "tel", "tel-country-code", "tel-extension", "tel-local", "tel-local-prefix", "tel-local-suffix", "tel-national", "transaction-amount", "transaction-currency", "url", "username", "work"] }, From 16d7eda192a41331666c744b973f6a4da75512ac Mon Sep 17 00:00:00 2001 From: Dan Date: Wed, 14 Sep 2016 15:32:40 -0400 Subject: [PATCH 11/11] Fixes missing comma --- src/extensions/default/HTMLCodeHints/HtmlAttributes.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/extensions/default/HTMLCodeHints/HtmlAttributes.json b/src/extensions/default/HTMLCodeHints/HtmlAttributes.json index 2762ebff2ba..70361f1fcb1 100644 --- a/src/extensions/default/HTMLCodeHints/HtmlAttributes.json +++ b/src/extensions/default/HTMLCodeHints/HtmlAttributes.json @@ -102,7 +102,7 @@ "cc-additional-name", "cc-csc", "cc-exp", "cc-exp-month", "cc-exp-year", "cc-family-name", "cc-given-name", "cc-name", "cc-number", "cc-type", "country", "country-name", "current-password", "email", "family-name", "fax", "given-name", "home", "honorific-prefix", "honorific-suffix", "impp", "language", "mobile", "name", - "new-password", "nickname", "off", "on" "organization", "organization-title", "pager", "photo", "postal-code", "sex", + "new-password", "nickname", "off", "on", "organization", "organization-title", "pager", "photo", "postal-code", "sex", "shipping", "street-address", "tel-area-code", "tel", "tel-country-code", "tel-extension", "tel-local", "tel-local-prefix", "tel-local-suffix", "tel-national", "transaction-amount", "transaction-currency", "url", "username", "work"] },