diff options
Diffstat (limited to 'ext')
| -rw-r--r-- | ext/js/background/backend.js | 2 | ||||
| -rw-r--r-- | ext/js/language/translator.js | 17 |
2 files changed, 11 insertions, 8 deletions
diff --git a/ext/js/background/backend.js b/ext/js/background/backend.js index 84ff2c0d..5b133d79 100644 --- a/ext/js/background/backend.js +++ b/ext/js/background/backend.js @@ -1896,7 +1896,7 @@ class Backend { return { wildcard, mainDictionary, - alphanumeric, + removeNonJapaneseCharacters: !alphanumeric, convertHalfWidthCharacters, convertNumericCharacters, convertAlphabeticCharacters, diff --git a/ext/js/language/translator.js b/ext/js/language/translator.js index dc91d9eb..856763a5 100644 --- a/ext/js/language/translator.js +++ b/ext/js/language/translator.js @@ -64,7 +64,7 @@ class Translator { * { * wildcard: (enum: null, 'prefix', 'suffix'), * mainDictionary: (string), - * alphanumeric: (boolean), + * removeNonJapaneseCharacters: (boolean), * convertHalfWidthCharacters: (enum: 'false', 'true', 'variant'), * convertNumericCharacters: (enum: 'false', 'true', 'variant'), * convertAlphabeticCharacters: (enum: 'false', 'true', 'variant'), @@ -169,8 +169,10 @@ class Translator { // Find terms internal implementation async _findTermsInternal(text, enabledDictionaryMap, options) { - const {alphanumeric, wildcard} = options; - text = this._getSearchableText(text, alphanumeric); + const {wildcard} = options; + if (options.removeNonJapaneseCharacters) { + text = this._getJapaneseOnlyText(text); + } if (text.length === 0) { return {dictionaryEntries: [], originalTextLength: 0}; } @@ -302,15 +304,16 @@ class Translator { return text; } - _getSearchableText(text, allowAlphanumericCharacters) { - if (allowAlphanumericCharacters) { return text; } + _getJapaneseOnlyText(text) { const jp = this._japaneseUtil; let length = 0; for (const c of text) { - if (!jp.isCodePointJapanese(c.codePointAt(0))) { break; } + if (!jp.isCodePointJapanese(c.codePointAt(0))) { + return text.substring(0, length); + } length += c.length; } - return length >= text.length ? text : text.substring(0, length); + return text; } _getTextOptionEntryVariants(value) { |