summaryrefslogtreecommitdiff
path: root/ext
diff options
context:
space:
mode:
authorStefanVukovic99 <stefanvukovic44@gmail.com>2023-12-29 04:02:51 +0100
committerGitHub <noreply@github.com>2023-12-29 03:02:51 +0000
commit580983b9b8a17965db1466aa08cad3c96b06022e (patch)
tree914760e326ffc4b1acf8ddcddbd5006ae762babf /ext
parent8d5d2152e4295fdcefa6ef283204c92df1f81305 (diff)
search resolution option (#436)
* add search resolution setting * move finding next substring to method * use regex literal * fix comments * fix comments * add options update function * update test
Diffstat (limited to 'ext')
-rw-r--r--ext/data/schemas/options-schema.json11
-rw-r--r--ext/js/background/backend.js4
-rw-r--r--ext/js/data/options-util.js15
-rw-r--r--ext/js/language/translator.js22
-rw-r--r--ext/settings.html12
5 files changed, 59 insertions, 5 deletions
diff --git a/ext/data/schemas/options-schema.json b/ext/data/schemas/options-schema.json
index f5b24160..65c4102e 100644
--- a/ext/data/schemas/options-schema.json
+++ b/ext/data/schemas/options-schema.json
@@ -728,9 +728,18 @@
"convertHiraganaToKatakana",
"convertKatakanaToHiragana",
"collapseEmphaticSequences",
- "textReplacements"
+ "textReplacements",
+ "searchResolution"
],
"properties": {
+ "searchResolution": {
+ "type": "string",
+ "enum": [
+ "letter",
+ "word"
+ ],
+ "default": "letter"
+ },
"convertHalfWidthCharacters": {
"type": "string",
"enum": ["false", "true", "variant"],
diff --git a/ext/js/background/backend.js b/ext/js/background/backend.js
index ae78a97b..68d4e0c8 100644
--- a/ext/js/background/backend.js
+++ b/ext/js/background/backend.js
@@ -2417,7 +2417,8 @@ export class Backend {
convertHiraganaToKatakana,
convertKatakanaToHiragana,
collapseEmphaticSequences,
- textReplacements: textReplacementsOptions
+ textReplacements: textReplacementsOptions,
+ searchResolution
}
} = options;
const textReplacements = this._getTranslatorTextReplacements(textReplacementsOptions);
@@ -2444,6 +2445,7 @@ export class Backend {
convertHiraganaToKatakana,
convertKatakanaToHiragana,
collapseEmphaticSequences,
+ searchResolution,
textReplacements,
enabledDictionaryMap,
excludeDictionaryDefinitions
diff --git a/ext/js/data/options-util.js b/ext/js/data/options-util.js
index c219bed3..eb9af9b6 100644
--- a/ext/js/data/options-util.js
+++ b/ext/js/data/options-util.js
@@ -555,7 +555,8 @@ export class OptionsUtil {
{async: false, update: this._updateVersion18.bind(this)},
{async: false, update: this._updateVersion19.bind(this)},
{async: false, update: this._updateVersion20.bind(this)},
- {async: true, update: this._updateVersion21.bind(this)}
+ {async: true, update: this._updateVersion21.bind(this)},
+ {async: false, update: this._updateVersion22.bind(this)}
];
/* eslint-enable no-multi-spaces */
if (typeof targetVersion === 'number' && targetVersion < result.length) {
@@ -1174,6 +1175,18 @@ export class OptionsUtil {
}
/**
+ * @type {import('options-util').ModernUpdateFunctionAsync}
+ */
+ _updateVersion22(options) {
+ // Added translation.searchResolution
+ for (const {options: profileOptions} of options.profiles) {
+ profileOptions.translation.searchResolution = 'letter';
+ }
+
+ return options;
+ }
+
+ /**
* @param {string} url
* @returns {Promise<chrome.tabs.Tab>}
*/
diff --git a/ext/js/language/translator.js b/ext/js/language/translator.js
index 733955c2..36ed8b43 100644
--- a/ext/js/language/translator.js
+++ b/ext/js/language/translator.js
@@ -329,8 +329,12 @@ export class Translator {
text2 = jp.collapseEmphaticSequences(text2, collapseEmphaticFull, sourceMap);
}
- for (let i = text2.length; i > 0; --i) {
- const source = text2.substring(0, i);
+ for (
+ let source = text2, i = text2.length;
+ i > 0;
+ i = this._getNextSubstringLength(options.searchResolution, i, source)
+ ) {
+ source = text2.substring(0, i);
if (used.has(source)) { break; }
used.add(source);
const rawSource = sourceMap.source.substring(0, sourceMap.getSourceLength(i));
@@ -343,6 +347,20 @@ export class Translator {
}
/**
+ * @param {string} searchResolution
+ * @param {number} currentLength
+ * @param {string} source
+ * @returns {number}
+ */
+ _getNextSubstringLength(searchResolution, currentLength, source) {
+ if (searchResolution === 'word') {
+ return source.search(/[^\p{Letter}][\p{Letter}\p{Number}]*$/u);
+ } else {
+ return currentLength - 1;
+ }
+ }
+
+ /**
* @param {string} text
* @param {TextSourceMap} sourceMap
* @param {import('translation').FindTermsTextReplacement[]} replacements
diff --git a/ext/settings.html b/ext/settings.html
index 323be708..1095e3b7 100644
--- a/ext/settings.html
+++ b/ext/settings.html
@@ -1505,6 +1505,18 @@
</div>
</div>
<div class="settings-group advanced-only">
+ <div class="settings-item advanced-only"><div class="settings-item-inner settings-item-inner-wrappable">
+ <div class="settings-item-left">
+ <div class="settings-item-label">Dictionary search resolution</div>
+ <div class="settings-item-description" lang="en"><p>"A dog"&#x3000;&rarr;&#x3000;search for "A dog","A do", "A d", "A"</p> or <p>"A dog"&#x3000;&rarr;&#x3000;"A dog", "A"</p></div>
+ </div>
+ <div class="settings-item-right">
+ <select data-setting="translation.searchResolution">
+ <option value="letter">Letter</option>
+ <option value="word">Word</option>
+ </select>
+ </div>
+ </div></div>
<div class="settings-item settings-item-button" data-modal-action="show,translation-text-replacement-patterns"><div class="settings-item-inner">
<div class="settings-item-left">
<div class="settings-item-label">Configure custom text replacement patterns&hellip;</div>