Day 1 inf #5
Day 1 inf #5
Conversation
palkan
left a comment
palkan
left a comment
There was a problem hiding this comment.
Есть пару моментов, которые надо исправить, а в целом всё верно.
app/views/tracks/index.html.erb
Outdated
| <% tracks.each_with_index do |track, i| %> | ||
| <%= render track, track_counter: page * page_size + i, enqueueble: true %> | ||
| <% end %> |
There was a problem hiding this comment.
Правильную нумерацию можно (и нужно) сделать через рендеринг коллекции; нужно просто задать стартовую позицию. Откатываться на each — плохая идея (это менее производительно).
There was a problem hiding this comment.
Поправила, правда я думала, что можно сделать через track_counter, но пришлось добавить новую локал переменную
app/views/tracks/_load_page.html.erb
Outdated
| @@ -0,0 +1 @@ | |||
| <%= turbo_frame_tag "tracks_page_#{page}", src: artist_tracks_path(artist, page: page), loading: "lazy" %> | |||
There was a problem hiding this comment.
Получается, что на каждую страницу мы рендеринг не только список треков, но и шапку (просто она не используем самим Турбо); получается лишняя и бесполезная работа для сервера. Нужно бы сделать так, чтобы возвращался только список треков (и следующий фрейм, если нужно).
2 participants
Задание:
Необходимо добавить новую страницу со всеми треками артиста с ленивой подгрузкой треков при прокрутке (aka infinite scroll pagination). Использовать только Turbo Frames.