Self code review

[utubo]

いったんクローズ

[utubo]

• [x] options.js L404-L407

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

  ↓

  const setupSettingItems = () => {

[utubo]

• [x] options.js L175 delete this line.

  b.classList.remove('hide');

[utubo]

• [x] 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]

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

  let exData = {
  customGestureList: []
  };

[utubo]

• [x] 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.*

• [x] adjustDlg, adjustBox →　adjustmentDlg

[utubo]

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

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

[utubo]

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

[utubo]

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

[utubo]

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

[utubo]

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

[utubo]

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

[utubo]

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

• [x] options.js
• [x] options.css
• [x] content.js

[utubo]

• [x] 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]

• [x] 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]

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

[utubo]

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

[utubo]

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

  a.parentNode.removeChild(a);

  ↓

  a.remove();

[utubo]

• [x] 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]

• [x] 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]

• [x] 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'); };

↓
```javascript
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]

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

[utubo]

• [x] background.js の最後

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

• [x] background.jsの コメント

  // Ignore invalid value

  ↓

  // Ignore the invalid return value.

• [x] content.js

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

  ↓

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

[utubo]

• [x] 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.
  }

• [x] 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へ持ち越し