Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

初级班 task0001 任务提交issue #764

Open
diysimon opened this issue Apr 27, 2015 · 69 comments
Open

初级班 task0001 任务提交issue #764

diysimon opened this issue Apr 27, 2015 · 69 comments

Comments

@diysimon
Copy link
Contributor

注意,请初级班的各位在本issue中,回复您的task0001Github仓库地址

@Melanie-BUPT
Copy link

Melanie-BUPT。task0001还没被review,所以再次提交试一试啦~地址:
https://github.com/Melanie-BUPT/ife/tree/master/task/task0001

@luxiaojian
Copy link

之前task0001还没被review,所以重新提交一次
https://github.com/luxiaojijan/ife/tree/master/task/task001/work/luxiaojijan

@caizone
Copy link

caizone commented Apr 28, 2015

之前没交上,麻烦各位大大了,多多提意见,谢谢。https://github.com/caizone/ife_task/tree/master/task0001

@Manyu17
Copy link

Manyu17 commented Apr 29, 2015

之前task0001还没被review,所以重新提交一次
https://github.com/UtopiaScript/ife/tree/master/task/task0001/work/UtopiaScript

@zmiaozzz
Copy link

task0001还没有被review,所以重新提交一次啦,谢谢~~~
https://github.com/zmiaozzz/ife/tree/master/task/task0001/work/zmiaozzz

@AlisonZhang
Copy link

task0001还没被review,所以重新提交,我是零基础刚入门的新手,代码写的比较混乱,给大师们增加负担了,希望能多多包涵。
https://github.com/AlisonZhang/ife/tree/master/task/task0001/work/AlisonZhang

@DIYgod
Copy link

DIYgod commented Apr 30, 2015

TASK 0001 重新提交:https://github.com/DIYgod/ife-work/tree/master/task0001

在线Demo: https://www.anotherhome.net/file/ife/task0001

问题记录(总结收获): https://www.anotherhome.net/1969

五一快乐,顺颂时祺。

@HansJiang
Copy link

task0001 重新提交:https://github.com/HansJiang/ife/tree/master/task/task0001/work/HansJiang
在线请戳:http://hansjiang.github.io/
我是靠自学的,有些可能不是很规范,命名方式也不是很好,请大牛们多多指导下。 花费你们五一玩耍时间了,不好意思。顺祝五一快乐。

@SIKE-MA
Copy link

SIKE-MA commented Apr 30, 2015

task0001重新提交,很期待大大们的review。五一快乐啦啦啦啦啦啦啦~
https://github.com/M-Withershins/ife/tree/master/task/task0001/work/M-Witherhins
在线demo:http://m-withershins.github.io/

@tingtingzhangbest
Copy link

task0001没被review,重新提交,多多包涵,辛苦了。
https://github.com/tingtingzhangbest/ife/tree/master/task/task0001/work/tingtingzhangbest

@ET-1992
Copy link

ET-1992 commented Apr 30, 2015

task0001重新提交,期待review,谢谢
https://github.com/die214144031/ife/tree/master/task/task0001/work/die214144031

@qqqhbbb
Copy link

qqqhbbb commented Apr 30, 2015

https://github.com/qqqhbbb/task/tree/master/task0001 期待指点,谢谢导师。

@Gaohaoyang
Copy link

TASK 0001 重新提交:

@oo6
Copy link

oo6 commented Apr 30, 2015

Task0001:

仅仅测试了Chrome,Firefox,Safari,ie11。

等待Review,千万不要笑话我代码和文章写的烂TAT,幼小的心灵受不了啊!!!

五一快乐,辛苦啦~~~

@zcp2123
Copy link

zcp2123 commented Apr 30, 2015

task0001重新提交,很期待大大们的review。五一快乐~~~~
https://github.com/zcp2123/ife/tree/master/task/task0001

@luckykaiyi
Copy link

https://github.com/lucky7ky/ife/tree/master/task/task0001/work/lucky7ky
在线Demo :http://ifetask1.luckykaiyi.com/
✨导师们辛苦了~ 五一快乐 ~

@ileenhow
Copy link

@lightbrush
Copy link

Task0001重新提交

非常非常非常感谢!真的很惭愧,我知道有些地方没有做好……我会加油改好的。

顺安
Light1980

@SupDream
Copy link

task0001 各位导师辛苦了,五一节快乐!
https://github.com/SupDream/ife/tree/master/task/task0001/work/SupDream

@ch1oechao
Copy link

task0001任务提交

导师辛苦啦 五一快乐~(^-^)~

@licop
Copy link

licop commented May 1, 2015

导师们辛苦了,希望自己以后也能成为大牛,帮助后辈们更好的学习

作业 https://github.com/licop/ife/tree/master/task/task0001/work/licop
博客总结 http://www.cnblogs.com/licop/

@reverland
Copy link

@candywxt
Copy link

candywxt commented May 1, 2015

@diysimon
Copy link
Contributor Author

diysimon commented May 1, 2015

@Melanie-BUPT review done, 注意email

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@luxiaojijan Review done,注意查收email

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@caizone Review done,见你github的issue

@wanluw
Copy link

wanluw commented May 2, 2015

辛苦导师们了

之前的 PR: #696
自己的 branch: https://github.com/wanluw/ife/tree/task0001/task/task0001/work/wanluw

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@UtopiaScript Review done,注意查收email

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@zmiaozzz Review done,注意查收email

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@AlisonZhang Review done,注意查收email

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@zcp2123 Review done,注意查收email。

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@candywxt review done,注意查收email

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@wanluw review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@shuaiyunzhang Review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 2, 2015

@reverland@Melanie-BUPT,小号?

@reverland
Copy link

@diysimon ?不是啊

我去。。。。。。看了看别人作业然后粘错了= =。。。。。。

已更正。。。

作业: https://github.com/reverland/ife/tree/master/task/task0001
在线: http://reverland.org/ife/task/task0001/work/reverland/

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@licop review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@zchen9 review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@lucky7ky review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@ileenhow Review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@Light1980 Review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@SupDream Review done,注意查收邮件

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@reverland Review done,注意查收邮件

@liuxuanzy
Copy link

@SupDream REVIEW结果如下:

通用

  1. 代码中存在标签对齐方式不一致的行为,另外有一些标签没有属性但是多了一个空格
  2. 这个页面怎么看都是一个中文页面,为什么lang要写成en呢,写zh好一点。
  3. 在页面中只有一个header的情况下,只接用header标签选择器就好了,不用再加一个类了吧,footer同样。
  4. 如果你认为页面的主要内容是一个section 那么页面的logo应当是h1而不是p,这里用a更好
  5. github图标应当使用a连接的bg来设置而不是加一个img标签,通常img标签用于表示一个图片是正文的一部分,而bg表示图片用于装饰,由于SEO相关会忽略CSS,所以用bg不会影响阅读,另外应当在a中间写上我的GITHUB之类的提示文字,并使用CSS隐藏,这样可以更好的进行SEO优化。
  6. 为什么有header而footer用了一个div。
  7. footer中的友情链接应当使用dl而不是ul,因为第一项和下边的内容有明显的定义关系。
  8. 版权信息那里同样应当使用bg.
  9. CSS中应当有一个通用的样式文件。
  10. img标签用到的图片和css中用到的图片应当用两个文件夹来存放,前者可能会随着服务器的变动发生变化,后者可能被放在CDN上。

index

  1. 要注意细心,html标签的lang属性的引号呢?
  2. 个人介绍也可以是一个article

blog

  1. 页面中的section没有必要。

gallery

  1. 上边的tab也是一个列表,用ul比较好。
  2. 看到你在页面中那个字符J用了s标签,可以解释一下这为什么用s标签么?
  3. 搜索框没有用form
  4. 人气排行是一个页面,另外应当在宽度的div里面加上数字,这样可以方便SEO

about

  1. 每一个项目可以用section来分类,要好过用div

总结

  1. 语义化做的还不错,但是在写CSS的时候很多因为语义化带来的遍历没有利用到
  2. 没有使用比较先进的编辑器
  3. 建议多进行实践,多对代码进行一些精益求精的处理。

@diysimon
Copy link
Contributor Author

diysimon commented May 3, 2015

@iuunhao
Copy link

iuunhao commented May 3, 2015

@liuyifeneve
Copy link

task0001还没有被review,重新提交一次,辛苦各位大大啦~~
https://github.com/liuyifeneve/ife/tree/master/task/task0001/work/liuyifeneve

@liuxuanzy
Copy link

@iuunhao

代码review的结果如下:

  1. 过分使用div和span,没有使用语义化的标签
  2. css命名过多使用top/center等无意义的词汇
  3. github图标应当加上一个文字说明,并使用css隐藏,以方面在CSS加载失败的时候内容可以正常显示。
  4. 页面中,每一个独立的文章的标题都应当从h1开始写。
  5. 书写一段文字应当使用p,而不要用div
  6. 下边的友情连接使用dl/dd/dt是不是会更好一些。
  7. 博客页的中的日历应当不是a,不具有链接特性。
  8. 人气排行榜使用ol更好,进度条中应当用隐藏的文字标注当前的进度。
  9. 页面中使用的是tab,尽量使用4个空格来缩进代码。
  10. 个人介绍页面中,不需要对内容进行展示上的分块,用流布局可能更方便。

@liuxuanzy
Copy link

@liuyifeneve

代码review意见如下:

  1. 尽量使用html的doctype,避免使用xhtml
  2. 所有的标签都没有缩进,请使用4个空格进行缩进
  3. 过分使用div和span,没有使用语义化的标签
  4. 在xhtml文档中,属性必须用引号进行闭合
  5. 没必要过多的使用浮动,可以适当使用绝对定位来解决问题,尤其是在头部这种定高的地方
  6. 清除浮动不要引入新标签,用:after配合zoom:1更好,减少html中的冗余代码。
  7. HTML和CSS应当分离,除非样式不能由前端决定,或者样式由JS或者后台计算得出,不要写行内样式。
  8. 尽量避免使用id选择器,多使用类选择器,不要给每一个标签都加一个id
  9. 下边的友情连接使用dl/dd/dt是不是会更好一些。
  10. 建议多阅读HTML和CSS的基础知识的书,多进行代码联系,你相对来说基础比较薄弱。

@Drupaloops
Copy link

之前的pull request没有被review,要交任务二才看到要在这里重新提交,求帮我看下
https://github.com/Drupaloops/ife/tree/master/task/task0001/work/Drupaloops

@julytian
Copy link

julytian commented May 4, 2015

@diysimon ,真的不好意思,之前的pull request没有被review,这是task0001任务, https://github.com/julytian/ife/tree/master/task/task0001/work

@luolala
Copy link

luolala commented May 5, 2015

重新提交一次,各位老师和同学五四快乐~
https://github.com/luolala/ife/tree/master/task/task0001/work/luolala

@kk7771
Copy link

kk7771 commented May 6, 2015

抽业余时间做了下,希望老师能看下 指点下我写的页面规范 布局 兼容性问题
https://github.com/liSNS/ife/tree/master/task/task0001/work

@anyever
Copy link

anyever commented May 6, 2015

刚刚才看到要在这提交。前辈们辛苦了!
https://github.com/anyever/ife/tree/master/task/task0001/work/anyever

@dodosophia
Copy link

dodosophia task0001
https://github.com/dodosophia/ife/tree/master/task/task0001
谢谢各位老师前辈~~~

@mengjie16
Copy link

之前task1没被review,在这里提交下,谢谢
https://github.com/mengjie16/ife/tree/master/task/task0001/work/mengjie16

@ZoeBai
Copy link

ZoeBai commented May 7, 2015

初级班task1任务,请老师抽空review,谢谢
https://github.com/ZoeBai/ife/tree/master/task/task0001/work/ZoeBai

@sg157
Copy link

sg157 commented May 24, 2015

麻烦帮忙看一下!虽然自己都不知道怎么看,感觉页面就是东拼西凑凑出来的!虽然知道不会被review但是还是完成了自己的一个心愿。github知道怎么用了
https://github.com/sg157/ife/tree/master/task/task0001/work/sg157

@blachworld
Copy link

感觉写的烂级了。
勉强提交一下,下次还要改改。
https://github.com/blachworld/task0001/tree/master/work/myGithubAccoutName

@Mzin
Copy link

Mzin commented Sep 12, 2019

计算机18-8杜龙龙
https://mzin.github.io/task0001/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests