search

f2etw / fedc2018

http://2018.fedc.tw/
3 stars 7 forks source link

feat(web): menu prototype #37

Closed wl00887404 closed 6 years ago

wl00887404 commented 6 years ago

修個 conflict ，等等

Rplus commented 6 years ago

其實我有改一些在自己的 branch 上 或許可以參考看看 style 的部份 https://github.com/Rplus/fedc2018/blob/0bb46ac6164ce8ca3cd5991b8d390fe946515535/style.scss

我已經放棄改 react 了，好累…

cjies commented 6 years ago

話說有修改的話，請個別留 commits 記錄，利於 review。 而不是整個 PR 只留一個 commit。

wl00887404 commented 6 years ago

@yutin1987 他教我的

cjies commented 6 years ago

比較好應該是等 review 完後一次過整理 commits 再 merge 吧 XD

Rplus commented 6 years ago

不用擔心 PR 裡太多 commit PR 這顆 image 會自動幫這次的 PR 壓成單一 commit 只是 PR 的 title 就會變成預設的 commit message 了 所以 title 要寫清楚處理哪些事

Jerry-Hong commented 6 years ago

@wl00887404 這個什麼時候好？ 我剛剛看漢堡選單的按鈕沒有換

wl00887404 commented 6 years ago

@Jerry-Hong 漢堡的動畫我不確定該長怎樣 Header 應該是已經好的 看哪邊需要修改

wl00887404 commented 6 years ago

我等等把多餘的地方去掉 應該很多東西會刪掉 我不保留 commit 行嗎？

@cjies BlurFitler 有其他作法嗎？ 不然我依然更動的 Index

@Jerry-Hong 請教漢堡的動畫

cjies commented 6 years ago

先以你的方式先搞定這 PR 吧。

DarkFilter 和 BlurFilter 如果現有版型可以 work 的話，可以不用急著改。 先讓網站可以上線為先 XD