Only sanitize words once when adding

FrostCo · Sep 9, 2018 · b36f0c4 · b36f0c4
1 parent e334c9a
commit b36f0c4
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 4 deletions.
diff --git a/dist/eventPage.js b/dist/eventPage.js
@@ -56,7 +56,6 @@ chrome.runtime.onMessage.addListener(function (request, sender, sendResponse) {
 // Add selected word/phrase and reload page (unless already present)
 function addSelection(selection) {
     return __awaiter(this, void 0, void 0, function* () {
-        selection = (selection.trim()).toLowerCase();
         let cfg = yield WebConfig.build(); // TODO: Only need words here
         let result = cfg.addWord(selection);
         if (result) {

diff --git a/dist/optionPage.js b/dist/optionPage.js
@@ -297,7 +297,7 @@ export default class OptionPage {
     }
     wordAdd(event) {
         let wordText = document.getElementById('wordText');
-        let word = wordText.value.trim().toLowerCase();
+        let word = wordText.value;
         let result = this.cfg.addWord(word);
         if (word != '') {
             if (result) {

diff --git a/src/eventPage.ts b/src/eventPage.ts
@@ -60,7 +60,6 @@ chrome.runtime.onMessage.addListener(
 //
 // Add selected word/phrase and reload page (unless already present)
 async function addSelection(selection: string) {
-  selection = (selection.trim()).toLowerCase();
   let cfg = await WebConfig.build(); // TODO: Only need words here
   let result = cfg.addWord(selection);
 

diff --git a/src/optionPage.ts b/src/optionPage.ts
@@ -308,7 +308,7 @@ export default class OptionPage {
 
   wordAdd(event) {
     let wordText = document.getElementById('wordText') as HTMLInputElement;
-    let word = wordText.value.trim().toLowerCase();
+    let word = wordText.value;
     let result = this.cfg.addWord(word);
 
     if (word != '') {