search

hoangphantich / eula.web16.1

Code repos. Homepage:
https://hoangphantich.github.io/webeula16.1/
1 stars 3 forks source link

[Review] Hồ Mậu Tuấn Linh #7

Open hoangphantich opened 7 years ago

hoangphantich commented 7 years ago

@linhho review 2 bạn này nhé @dongvo Võ Đình Đông|1633771356|https://www.facebook.com/DVDongVo|||dinhdong19995 @cuongcau93 Nguyễn Mạnh Cường|1647543108|https://www.facebook.com/cuongcau93|||deptrai_hocgioi_khongcankhena@yahoo.com.vn

linhhoo commented 7 years ago

@cuongcau93

Tổng điểm: 8đ.

screen shot 2016-10-07 at 09 17 35 screen shot 2016-10-07 at 09 13 09
linhhoo commented 7 years ago

@dongvo Không thấy source code của em Đông

hoangphantich commented 7 years ago

@dongvo, @linhho https://github.com/dongvo/FrontEnd/tree/master

linhhoo commented 7 years ago

@dongvo

  1. Form: 3/5
    • Username yêu cầu nhập ít nhất 6 kí tự nhưng nhập 5 kí tự thì đã k còn thông báo lỗi nữa.
    • Password nhập testcase 'aA1@' vẫn báo lỗi.
    • Email và re pepassword đã hoạt động đúng.
    • Nên check sẵn giới tính và chọn sẵn 1 quốc giá, nếu không check sẵn thì cần có validate.
  2. UI: (0.5/1)
    • Cần thống nhất ngôn ngữ, không nên vừa tiếng anh vừa tiếng việt.
    • Sử dụng màu nền xanh thẫm thì nên cho chữ màu trắng.
  3. UX: tốt (1/1)
  4. Code: 1.5/2
    • Nên sử dụng IIFE
    • perfomance and security: 1/1

Total: 7/10

dongvo commented 7 years ago

@linhho password e để là phải 8 kí tự . em viết thiếu cái thông báo đấy phần user em nhầm thành 5 kí tự, để em commit lại

linhhoo commented 7 years ago

@cuongcau93 Lap Udemy: Giao diện chưa đẹp lắm, nên tách javascript thành 1 file riêng. Chưa có chức năng đăng nhập bằng google hoặc facebook account. Chưa mã hóa cho mật khẩu. 8đ.

linhhoo commented 7 years ago

@dongvo Lap Udemy: Giao diện chưa có chuyển đổi giữa 2 hình nền, chưa có khung search màu đen bên trên hình nền. Chưa có chức năng đăng nhập bằng google hoặc facebook account. Chưa mã hóa cho mật khẩu. 8đ