alex/yomichan
1
Fork 0
Code Releases Activity

Replace and remove old character/string testing functions

Browse Source
This commit is contained in:
toasted-nutbread 2020-01-22 21:53:47 -05:00
parent 1fa8a59626
commit 317bf35bc0
4 changed files with 6 additions and 52 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ext/bg/js/handlebars.js
View File

@ -61,7 +61,7 @@ function handlebarsFuriganaPlain(options) {
function handlebarsKanjiLinks(options) {
let result = '';
for (const c of options.fn(this)) {
if (jpIsKanji(c)) {
if (jpIsCharCodeKanji(c.charCodeAt(0))) {
result += `<a href="#" class="kanji-link">${c}</a>`;
} else {
result += c;

2
ext/bg/js/search.js
View File

@ -265,7 +265,7 @@ class DisplaySearch extends Display {
text !== this.clipboardPreviousText
) {
this.clipboardPreviousText = text;
if (jpIsAnyCharacterJapanese(text)) {
if (jpIsStringPartiallyJapanese(text)) {
this.setQuery(this.isWanakanaEnabled() ? window.wanakana.toKana(text) : text);
window.history.pushState(null, '', `${window.location.pathname}?query=${encodeURIComponent(text)}`);
this.onSearchQueryUpdated(this.query.value, true);

3
ext/bg/js/translator.js
View File

@ -590,8 +590,7 @@ class Translator {
if (!options.scanning.alphanumeric) {
const ii = text.length;
for (let i = 0; i < ii; ++i) {
const c = text[i];
if (!jpIsCharacterJapanese(c)) {
if (!jpIsCharCodeJapanese(text.charCodeAt(i))) {
text = text.substring(0, i);
break;
}

51
ext/mixed/js/japanese.js
View File

@ -162,52 +162,6 @@ function jpIsStringPartiallyJapanese(str) {
}
// Old character/string testing functions
function jpIsKanji(c) {
const code = c.charCodeAt(0);
return (
(code >= 0x4e00 && code < 0x9fb0) ||
(code >= 0x3400 && code < 0x4dc0)
);
}
function jpIsKana(c) {
const code = c.charCodeAt(0);
return (
(code >= 0x3041 && code <= 0x3096) || // hiragana
(code >= 0x30a1 && code <= 0x30fc) // katakana
);
}
function jpIsCharFullWidth(c) {
const code = c.charCodeAt(0);
return (
(code >= 0xff21 && code <= 0xff3a) || // full width upper case roman letters
(code >= 0xff41 && code <= 0xff3a) || // full width upper case roman letters
(code >= 0xff10 && code <= 0xff19) // full width numbers
);
}
function jpIsKanaHalfWidth(c) {
const code = c.charCodeAt(0);
return (code >= 0xff66 && code <= 0xff9f); // half width katakana
}
function jpIsCharacterJapanese(c) {
return jpIsKanji(c) || jpIsKana(c) || jpIsCharFullWidth(c) || jpIsKanaHalfWidth(c);
}
function jpIsAnyCharacterJapanese(text) {
for (const c of text) {
if (jpIsCharacterJapanese(c)) {
return true;
}
}
return false;
}
// Conversion functions
function jpKatakanaToHiragana(text) {
@ -250,7 +204,7 @@ function jpConvertReading(expressionFragment, readingFragment, readingMode) {
if (readingFragment) {
return jpToRomaji(readingFragment);
} else {
if (jpIsKana(expressionFragment)) {
if (jpIsStringEntirelyKana(expressionFragment)) {
return jpToRomaji(expressionFragment);
}
}
@ -307,7 +261,8 @@ function jpDistributeFurigana(expression, reading) {
const groups = [];
let modePrev = null;
for (const c of expression) {
const modeCurr = jpIsKanji(c) || c.charCodeAt(0) === 0x3005 /* noma */ ? 'kanji' : 'kana';
const charCode = c.charCodeAt(0);
const modeCurr = jpIsCharCodeKanji(charCode) || charCode === JP_ITERATION_MARK_CHAR_CODE ? 'kanji' : 'kana';
if (modeCurr === modePrev) {
groups[groups.length - 1].text += c;
} else {