test
カーソルを変更する処理を外に出した。処理自体は未成功。
setAttributeをプロパティに変更

HTMLオブジェクトの属性を変更する場合直接プロパティを変更してやればいい
普通のオブジェクトでプロパティ名にハイフンがあるとこのメソッドが必要
classの追加にclassList.addを使う

classは複数設定できるがこの方法なら安全に追加できる
直接文字列で指定する場合はclassNameプロパティが使用できる
createTextNodeを使わずにtextContentプロパティで直接指定する

Textはオブジェクトなので色々プロパティを持っているが追加するだけならtextContentで十分
Merge pull request #1 from nobby451/correction

Correction
ブックマークでulとliの親子関係がうまくできてなかったので修正。折りたたみ追加。\n備忘用に、イベント伝播検証用のJSコミット
表示を変更
ブックマークツリーの生成を若干すっきりさせる

Array.reduceを使い、一回の配列走査で効率よくliの入ったulを生成する
addEventListenerはliの生成時に一緒に追加する
もはやidは必要ないのでつけないようにする

ulを生成してidをセットする処理をなくす
ulを生成してreduceするところまでをfunctionにまとめる
空ディレクトリーの下にはulが生成されないようにする
Merge pull request #2 from nobby451/correction

Correction
Merge branch 'master' of https://github.com/marrontan619/JS
チェックボックスの状態を保持する方法を調べてみた
manifest.jsonから不要なパーミッションを削除

extensionは有効なキーワードではない
BookmarksにはtabsとactiveTabは不要
oneClickSaveでchrome.tabs.executeScriptを実行するのに必要なのはactiveTabのみ
冗長な記述を削除

三項演算子の条件はプロパティーを取ってきてるだけなので括弧は不要
ifの条件式はtrueと==で比較しているので二重に比較していることになってしまう
===の比較であれば型も一致していないとtrueにならないので意味がある
Merge pull request #3 from nobby451/refactoring

Refactoring
cancel
作業中：sendMessage()はJSONで送られるらしく、html要素を入れると怒られる
整理のためのコミット。