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

Tăng tốc độ tải trang #179

Open
2 of 4 tasks
thien-do opened this issue Oct 30, 2020 · 10 comments
Open
2 of 4 tasks

Tăng tốc độ tải trang #179

thien-do opened this issue Oct 30, 2020 · 10 comments
Assignees
Labels
enhancement improvement for existing feature medium priority

Comments

@thien-do
Copy link
Contributor

thien-do commented Oct 30, 2020

Mình nghĩ với dự án này thì việc tải trang nhanh, hay nói chính xác hơn là việc tải được nhiều thông tin nhất trong khoảng thời gian ngắn nhất, sẽ có lợi cho người dùng do đặc điểm của đối tượng người dùng bên mình.

Mình có kiểm tra và nhận thấy có 1 số điểm chúng ta có thể tối ưu được:

  • Sử dụng font có sẵn thay vì custom web font. Dù hiện tại đã dùng "swap" nhưng nó vẫn tốn khoảng thời gian fetch css của font (do file css lấy từ google fonts, và afaik google fonts có dựa vào request user agent để trả về css phù hợp)
  • Tối ưu hình ở trang chủ. Mình không chắc hình có được tối ưu chưa, nhưng cái rõ hơn là kích thước khá lớn. Nếu người dùng vào bằng điện thoại (mình nghĩ là phổ biến) thì kích thước này khá là dư thừa.
  • Có 1 cái loading lúc ban đầu, mình k rõ là loading gì. Nhưng minh có để ý là khoảng thời gian để server trả lời cũng đã khá đáng kể (TTFB), nhưng vẫn có client loading. Mình sẽ tìm hiểu kỹ hơn về mục này.
  • Sử dụng native thay vì jquery Không sử dụng jquery để giảm thời gian tải trang #181
@thien-do
Copy link
Contributor Author

cc @kc97ble

@DungDA
Copy link
Collaborator

DungDA commented Oct 30, 2020

@dvkndn �cảm ơn bạn nhiều nhé, những thứ chi tiết và tỉ mỉ này cần có người giúp, bạn có thể tạo PR hoặc sửa cùng team nhé, ngoài ra thêm được lazy load cho ảnh và video trên site hiện tại được thì tốt quá.

@thien-do
Copy link
Contributor Author

ồ mình chỉ mới thấy ảnh chứ chưa thấy video. Ít nhất với tấm hình ở trang chủ thì mình nghĩ có thể lazy vì nội dung nó không critical cho công tác cứu hộ.

Để mình tạo PR cho 2 mục trên :D

@DungDA
Copy link
Collaborator

DungDA commented Oct 30, 2020

@dvkndn hi trang này có video: https://cuuhomientrung.info/huong_dan_tnv/

Tạo code theo kiểu tiện cho việc sau này luôn, ví dụ các bạn sau muốn dùng lại lazy load chỉ việc theo 1 format là content tự nhận. Và vụ font chữ với tối ưu cũng tạo chuẩn sẵn để các bạn làm về sau follow theo bạn luôn (y)

@thien-do
Copy link
Contributor Author

Ok bạn :D

@dothanhlam97
Copy link
Collaborator

cc a @mihnsen@kc97ble

@thien-do
Copy link
Contributor Author

tks @dothanhlam97 . Anh lỡ cc @kc97ble khắp các mặt trận luôn rồi :)) chắc mai em nó nhiều noti lắm

@mihnsen
Copy link
Collaborator

mihnsen commented Oct 31, 2020

cc a @mihnsen@kc97ble

All good for me!

@mihnsen
Copy link
Collaborator

mihnsen commented Oct 31, 2020

@dvkndn
Để mình thêm ít thông tin, để chia nhỏ xuống xem có dễ hơn ko?
2 cái trang sau đây là public ai cũng vào xem được. - có thông tin người dùng, đang dùng webpack để build js/css

Những trang được liệt kê ở đây là template trong django admin, sẽ có màn hình authenticate để vào các trang này sớm (theo mình nghĩ vậy).
image

Về 2 cái checkbox cần làm mình chia thành 2 issues nhỏ để mention những ng làm nó từ đầu rồi ta resolve luôn nha:

@thien-do
Copy link
Contributor Author

thien-do commented Nov 1, 2020

cám ơn @mihnsen nhé

về mức độ ưu tiên mình nghĩ là 2 trang public quan trọng nhất (tại vì expected là users sẽ truy cập trang trong điều kiện khó khăn) còn những trang admin có thể tối ưu sau (vì mình nghĩ là admin sẽ có điều kiện mạng tốt hơn public users?)

@kc97ble kc97ble added enhancement improvement for existing feature high priority labels Nov 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improvement for existing feature medium priority
Projects
None yet
Development

No branches or pull requests

5 participants