-
Notifications
You must be signed in to change notification settings - Fork 50
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
Comments
cc @kc97ble |
@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á. |
ồ 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 |
@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) |
Ok bạn :D |
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 |
@dvkndn 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). 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:
|
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?) |
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:
The text was updated successfully, but these errors were encountered: