Revert "modular options, eslint, notyf"

softcup commented 7 years ago

Reverts softcup/New-Tongwentang-for-Firefox#23

options.html 內需要的 css 跟 js 漏掉了，麻煩 @t7yang 修正後再重新 pull request

t7yang commented 7 years ago

我這裡build出來的結果看起來是好的 @_@ 你那裏build出來會的套件會爛掉嗎？好，我再檢查一下。

softcup commented 7 years ago

package.json 我剛剛有重新 commit 了

t7yang commented 7 years ago

甚麼意思，我有不小心異動到package.json不對的地方嗎？

剛剛再build一次出來確認，options頁面正常。

是指我拿掉 options.html 裡面的 css 和 js 連結嗎？是的話，那是因為 webpack 會自動注入到 options.html 中

抱歉，還在摸索學習中，有做錯的地方還麻煩多跟我解釋一下，不一定可以意會。

softcup commented 7 years ago

我這邊也正常了，剛剛不曉得為什麼沒有自動注入，目前看來解決了。

t7yang commented 7 years ago

瞭解，還好沒事。

softcup commented 7 years ago

package.json 是被我異動到，我這邊的問題，還不太會直接在 github 上操作檔案。

t7yang commented 7 years ago

瞭解，麻煩了。

softcup commented 7 years ago

如果 @t7yang 有空的話，可以補一下 yarn run build 跟 yarn run dev 的詳細說明，各自做什麼用的。謝謝。

t7yang commented 7 years ago

沒問題，我會把readme.md寫得更詳細。

softcup commented 7 years ago

我剛剛正常的原因是，沒有 marge 前的程式，所以都正常，拉出 merge 後的程式後，在做 build 時，會有錯誤訊息，導致動作沒完成，所以新同文堂執行會出現錯誤。

softcup commented 7 years ago

[0] ./src/background.js 1.82 kB {2} [built] [failed] [1 error] [1] ./src/lib/options/options.js 1.82 kB {1} [built] [failed] [1 error] [2] ./src/lib/tongwen/tongwen.js 1.82 kB {0} [built] [failed] [1 error]

ERROR in ./src/background.js Module build failed: Error: Cannot find module 'debug'

以下省略，都是這樣的錯誤

t7yang commented 7 years ago

dist.zip 是喔，這就有點奇怪了。這裡是我這裡build出來的結果，麻煩你先試試看。我這裡試試看重新 git clone 一份下來看是不是真的是我弄壞了。

softcup commented 7 years ago

好的，麻煩你了。

softcup commented 7 years ago

測試你的 dist.zip 結果是正常的，這樣可能需要確認彼此的環境差異了。

t7yang commented 7 years ago

剛剛測試從 dev branch 直接 git clone 後 build 的結果是完全正確的（但有發現其他的問題就是，匯入所有選項時 notyf 不會顯示訊息。 url filter 的檢查機制需要修改）

可以麻煩你有空的時候測試直接從我的 dev branch 下載然後 build 嗎？ https://github.com/t7yang/New-Tongwentang-for-Firefox/tree/develop 如果這個的結果是可行的話，那我懷疑問題有可能是接受PR時造成的。

softcup commented 7 years ago

$> npm install debug 安裝的 debug 套件後，就正常了

t7yang commented 7 years ago

嗯，那不曉得是不是 yarn 跟 npm 的關係。

softcup commented 7 years ago

或許有可能。另外，webpack 能調整 css 注入的位置嗎？想將它調整到 head 的區塊中，這樣在載入設定畫面時，才比較不會看到沒套版面的設定畫面。

t7yang commented 7 years ago

預設就是注入到<head>裡頭，只是因為是用 JS 注入的關係，所以是等 JS 載入後才注入樣式。

我這裡是幾乎快到的看不到沒有樣式的情況，但要改善應該也沒有很難，加掛一些 webpack 外掛應該是可以做到的。

我把 url filter 、 notyf 、樣式的部分再推一個 PR 給你，不過可能要等幾天或是這個週末才有辦法。

build 的部分，可以的話， @ettoolong 也協助測試看看，感謝。

softcup commented 7 years ago

@t7yang 這個能不能不要透過 js 來注入，直接放在 html 的 head 內呢？

等你有空再推 PR 給我就好。

tongwentang / New-Tongwentang-for-Firefox

Revert "modular options, eslint, notyf" #24