Open petercpg opened 9 years ago
:+1:
let me do this
On 2015年3月19日週四 9:26 Bo-Yi Wu notifications@github.com wrote:
[image: :+1:]
— Reply to this email directly or view it on GitHub https://github.com/moztw/www.moztw.org/issues/198#issuecomment-83248407.
可能還有其他的 lib 有重複唷,這些資料夾裡的 jquery 說不定用同一個版本也都能動,如果可以的話測一下會更好。
有個 assets repo 就是要作這件事的啊
要移去 assets 也行,總之不要各頁塞一個我想比較好
從我的 iPod 傳送
Chen-Heng Chang notifications@github.com 於 2015年3月19日 10:55 寫道:
有個 assets repo 就是要作這件事的啊
— Reply to this email directly or view it on GitHub.
所以有 assets ,也有 lib 的意思? 好哦我把所有 code 先 review 一次, 希望能確保不同版本的 jQuery 看看能不能換同一個, 不能換的就先不動它然後一樣列這個 issue 裡面之後修
On 2015年3月19日週四 11:14 Peter Chen notifications@github.com wrote:
要移去 assets 也行,總之不要各頁塞一個我想比較好
從我的 iPod 傳送
Chen-Heng Chang notifications@github.com 於 2015年3月19日 10:55 寫道:
有個 assets repo 就是要作這件事的啊
— Reply to this email directly or view it on GitHub.
— Reply to this email directly or view it on GitHub https://github.com/moztw/www.moztw.org/issues/198#issuecomment-83291236.
不要分 Repo(過度複雜化),先從 jQuery 開始就好。
@kidwm 要 propose 任何架構改變前,都請先準備好文件,不然步驟只留在你腦中,別人無從跟進。
@mingtsay 你台灣版那個 bug 還沒結束,先做完再作第二個吧…
以下為整理好的各個網頁用到的 library 列表,先記著之後處理比較方便:
以下為整理好的 repo 內的 library 列表,先記著之後處理比較方便:
also solve #188 with this issue
@irvin 你不想分 repo 本來就不可能跟進啊 XD
@mingtsay 幫忙把 library 各自開 bug (eg. 合併各頁自帶的 jQuery、合併各頁自帶的 leaflet.js ),然後全部設一個 milestone,我們來整理
issue opened: #226, #227, #228, #229, #230, #231, #232
repo 裡某些活動或專案網頁都有自己放一份 library,可以統一合併到 /js/library。
隨便列舉一個 jquery 跟相關 plugin petercpg@v3:~/moztw-repo/moztw_web/www.moztw.org$ find ./ -name jq* ./twoffice/jquery-1.6.4.min.js ./twoffice/fancybox/jquery.fancybox-1.3.4.js ./twoffice/fancybox/jquery.fancybox-1.3.4.pack.js ./twoffice/fancybox/jquery.mousewheel-3.0.4.pack.js ./twoffice/fancybox/jquery.fancybox-1.3.4.css ./twoffice/fancybox/jquery.easing-1.3.pack.js ./foxmosa/js/jquery.min.js ./events/sopa/assets/jquery.js ./events/moztw-lab/assets/jquery.js ./events/app-dev-party/assets/jquery.js ./events/firefox4party/landing/jquery.wordcloud.min.js ./events/firefox4party/jquery.wordcloud.min.js ./events/scp0712/assets/jquery.js ./events/code-rush/assets/jquery.js ./js/library/jquery.lightbox-0.5.css ./js/library/fancybox2/jquery.fancybox.pack.js ./js/library/fancybox2/helpers/jquery.fancybox-thumbs.js ./js/library/fancybox2/helpers/jquery.fancybox-buttons.js ./js/library/fancybox2/helpers/jquery.fancybox-buttons.css ./js/library/fancybox2/helpers/jquery.fancybox-thumbs.css ./js/library/fancybox2/jquery.fancybox.js ./js/library/fancybox2/jquery.fancybox.css ./js/library/jquery.lightbox-0.5.pack.js ./js/library/jquery.lightbox-0.5.js ./js/library/jquery.lightbox-0.5.min.js ./js/library/jquery.js ./contribute/css/ui-lightness/jquery-ui-1.7.3.custom.css ./contribute/js/jquery-ui-1.7.3.custom.min.js ./contribute/js/jquery-1.3.2.min.js