Self code review

[utubo]

No description provided.

[utubo]

いったんクローズ

[utubo]

• options.js L404-L407

const setupSettingItems = res => {
		if (res && res.simple_gesture) {
			SimpleGesture.ini = res.simple_gesture;
		}

↓

const setupSettingItems = () => {

[utubo]

• options.js L175
  delete this line.

b.classList.remove('hide');

[utubo]

• options.css L18
  Title's padding is may be .8em 1em.

.title {
	background: #f2f2f2;
	border-bottom: 2px solid #0099ff;
	font-weight: bold;
	padding: 1em; ←HERE
}

[utubo]

• options.js L35-L37
  1行にしちゃおうかな

let exData = {
	customGestureList: []
};

[utubo]

• background.js L94
  typo. linNumber -> lineNumber

const code = `alert('${msg}');`; // TODO: Always e.lieNumber is 0.

[utubo]

• background.jsのこれベッタベタなのでなんとかしたい…

e.message.indexOf('result is non-structured-clonable data') === -1 // Ignore invalid value.

e.fileName: resource://gre/modules/ExtensionContent.jsm →alertしたい例外(syntax errorもこっち)
e.fileName: <anonymous code> →問題の例外(他にもこっちになるのがあるかも？)

もっとしっかりした切り分けはないのか

→ Season 2に持ち越し

[utubo]

options.*

• adjustDlg, adjustBox →　adjustmentDlg

[utubo]

• options.js この行document.getElementsByClassName使えば消せる

const TEXT_FORMS = ['newTabUrl', 'userAgent', 'timeout', 'strokeSize'];

[utubo]

• やっぱりActionじゃなくてGestureかなぁ

[utubo]

• options.js TEXT_FORMS 名前が変だから直したいなぁ何がいいかな
  →適当に直した

[utubo]

parentByClass → getParentByClass かなぁと思ったけどjQueryとかだとparent()なんだよな…

[utubo]

• container→itemかなぁ？
• ページのリンクはAタグでやったほうがいいしそうすればタッチ中の色付けも簡単そう
• ダイアログがでるitemやボタンはBUTTONタグの方がいいかな？
  →BUTTON使えやってかいあるわ… https://developer.mozilla.org/ja/docs/Web/HTML/Element/a
  →BUTTONにするとCSSの定義が長くなるのでDIVでいいや
  →:activeは反応が遅くてダメだった

[utubo]

• options.css
  emよりもremの方が良さそう

[utubo]

トーストつくったらまたソースがヤバイことになってしまった

• options.js
• options.css
• content.js

[utubo]

• options.js L.506-514

byId('index').addEventListener...
byId('index').addEventListener...
byId('index').addEventListener...

↓

const index = byId('index);
index.addEventListener...
index.addEventListener...
index.addEventListener...

[utubo]

• options.js
  saveTextValues → saveBindingValues
saveTextValuesDleay → saveBindingValuesDelay

[utubo]

touchstartとtouchmoveはasyncにしたほうがいいかも？
touchendはpreventDefaultするのでasyncはダメ
↓
結論：このままでOKいいや。
ためしてみた。
https://utb.sakura.ne.jp/samples/touchmove_interval.html
アドオンを無効にしても対して変わらない。
asyncにしたらむしろ重くなったまである

この件別issueにしとこうかな
→ #39

[utubo]

• content.js
  lastInnerWidth が一生「0」！

[utubo]

バージョンの上げ方間違ったわ…
2.5.5→2.6の間違いだわ…

[utubo]

• options.js 2か所くらいあるっぽい

a.parentNode.removeChild(a);

↓

a.remove();

[utubo]

• options.js
  let→const

let s = byId('addCommandToScript'); ←
let f = document.createDocumentFragment(); ←
f.appendChild(s.firstChild.cloneNode(true));
for (let i of allByClass('gesture-item')) {
	let name = i.id.replace(/_item/, ''); ←
	if (name === dlgs.editDlg.targetId) continue;
	if (!name) continue;
	let o = document.createElement('OPTION'); ←
	o.value = name;
	o.textContent = byClass(i, 'gesture-caption').textContent;
	f.appendChild(o);
}

const addCommand = e => {
let name = e.target.value; ←
if (!name) return;
let script = `\nSimpleGesture.doCommand('${name}');\n`;
if (name[0] === CUSTOM_GESTURE_PREFIX) {
	let title = findCustomGesture(name).title.replace(/\*/, ' '); ←
	script = `/* ${title} */${script}\n`;
}

// edit text values --
const saveBindingValues = e => {
	clearTimeout(TIMERS.saveBindingValues);
	for (let elm of bidingForms) {
		let ini = elm.classList.contains('js-binding-exData') ? exData : SimpleGesture.ini; ←

for (let elm of bidingForms) {
	let ini = elm.classList.contains('js-binding-exData') ? exData : SimpleGesture.ini; ←

[utubo]

• options.js

const addCommand = e => {
let name = e.target.value; ←
if (!name) return;
let script = `\nSimpleGesture.doCommand('${name}');\n`;
if (name[0] === CUSTOM_GESTURE_PREFIX) {
	let title = findCustomGesture(name).title.replace(/\*/, ' ');
	script = `/* ${title} */${script}\n`;
}

↓

const addCommand = e => {
let name = e.target.value; ←
if (!name) return;
let script = `\nSimpleGesture.doCommand('${name}');\n`;
if (name[0] === CUSTOM_GESTURE_PREFIX) {
	let title = findCustomGesture(name).title.replace(/\/\*|\*\//g, '  '); ←
	script = `\n/* ${title} */${script}\; ←
}

[utubo]

• options.js

const findCustomGesture = id => {
	return exData.customGestureList.find((e, i, a) => e.id === id);
};

const ifById = id => (typeof id === 'string') ? byId(id): id;
const fadeout = elm => { ifById(elm).classList.add('transparent'); };
const fadein = elm => { ifById(elm).classList.remove('transparent'); };

↓

const findCustomGesture = id => exData.customGestureList.find((e, i, a) => e.id === id);
const ifById = id => (typeof id === 'string') ? byId(id): id;
const fadeout = elm => ifById(elm).classList.add('transparent');
const fadein = elm => ifById(elm).classList.remove('transparent');

[utubo]

• トーストのフォントサイズは5vminでよいのでは🤔
  →拡大表示してるとトーストも拡大されちゃうのでNG

[utubo]

• background.js の最後

arg.tab = tab || (await browser.tabs.query({ active: true, currentWindow: true })[0];

• background.jsの コメント

// Ignore invalid value

↓

// Ignore the invalid return value.

• content.js

SimpleGesture.doCommand = (g, o) => ...

↓

SimpleGesture.doCommand = (g, options) => ...

[utubo]

• exec.open

if (code) {
	const remove = () => { browser.tabs.onUpdated.removeListener(f); };
	const timer = setTimeout(remove, 5000); // when the tab doesn't complete.
	const f = (tabId, changeInfo, tab) => {
		if (changeInfo.status !== 'complete' || tabId !== arg.tab.id) return;
		clearTimeout(timer);
		remove();
		exec.executeScript({ tabId: tabId, code: code});
	};
	browser.tabs.onUpdated.addListener(f); // Firefox for Android doesn't support extraParameter.tabId.
}

• exec.executeScript

browser.tabs.executeScript(arg.tabId, { code: userScript }).then(result => {
	result && result[0] && result[0].url && exec.open(result[0]);
}).catch(e => {
	if (e.message === 'SimpleGestureExit') return;
	if (e.message.indexOf('result is non-structured-clonable data') !== -1) return; // Ignore the invalid return value.
	const msg = e.message.replace(/(['\\])/g, '\\$1');
	const code = `alert('${msg}');`; // TODO: Always e.lineNumber is 0.
	browser.tabs.executeScript({ code: code });
});

[utubo]

このIssueはクローズして、残りはSeason 2へ持ち越し