diff --git a/app/views/attendance_tracker/_course_table.erb b/app/views/attendance_tracker/_course_table.erb index 732e508..1fd4e85 100644 --- a/app/views/attendance_tracker/_course_table.erb +++ b/app/views/attendance_tracker/_course_table.erb @@ -16,12 +16,11 @@ <%# TODO: Improve this figure, atm it is for lectures in week, not time spent in week (ie prepping for lecture next week will count in next week not current week) %> <% if total_for_week > 0 %> - (Total <%= humanise_duration(lectures.sum { |lecture| lecture.total_overall_time }.seconds) %> - ) + (Total <%= humanise_duration(lectures.sum { |lecture| lecture.total_overall_time }.seconds) %>) <% end %> - <%= render lectures, { div: true } %> + <%= render lectures %> <% end %> diff --git a/app/views/attendance_tracker/today.html.erb b/app/views/attendance_tracker/today.html.erb index 7c94a15..1dc025d 100644 --- a/app/views/attendance_tracker/today.html.erb +++ b/app/views/attendance_tracker/today.html.erb @@ -10,7 +10,7 @@
Action
- <%= render @lectures, { div: true, today_view: true } %> + <%= render @lectures, { today_view: true } %> diff --git a/app/views/lectures/_lecture.html.erb b/app/views/lectures/_lecture.html.erb index e34406c..7c8aea0 100644 --- a/app/views/lectures/_lecture.html.erb +++ b/app/views/lectures/_lecture.html.erb @@ -1,69 +1,26 @@ -<% if local_assigns[:div] %> - <% status_classes = class_names({ - 'lecture-future': lecture.start_time.future?, - 'bg-purple-100': lecture == @current_lecture, - 'bg-green-100': lecture.attended?, - }) %> +<% status_classes = class_names({ + 'lecture-future': lecture.start_time.future?, + 'bg-purple-100': lecture == @current_lecture, + 'bg-green-100': lecture.attended?, + }) %> -
-
- <%= lecture.title %> +
+
+ <%= lecture.title %> - <% if lecture.online %> - - <% end %> -
- -
- <% if local_assigns[:today_view] %> - <%= lecture.start_time.to_fs(:time) %> ‐ <%= lecture.end_time.to_fs(:time) %> - <% else %> - <%= lecture.start_time.to_fs(:dmy) %> - - - <% end %> -
- -
- <%= render partial: 'lectures/lecture_status_icons', locals: { lecture: } %> -
- -
- <%= render partial: 'lectures/lecture_action', locals: { lecture: } %> -
- -
- <% if lecture.recording %> - <%= link_to "Open recording", lecture.recording&.recording_url %> - <% end %> -
+ <% if lecture.online %> + + <% end %>
-<% else %> - - - <%= lecture.title %> - <% if lecture.online %> - - <% end %> - - - +
+ <% if local_assigns[:today_view] %> + <%= lecture.start_time.to_fs(:time) %> ‐ <%= lecture.end_time.to_fs(:time) %> + <% else %> <%= lecture.start_time.to_fs(:dmy) %> - + <% end %> +
- - <%= render partial: 'lectures/lecture_status_icons', locals: { lecture: } %> - +
+ <%= render partial: 'lectures/lecture_status_icons', locals: { lecture: } %> +
- - <% joinable_time = lecture.start_time - 5.minutes %> - <% if joinable_time.future? %> - <%= button_to "Prepare", - lecture_start_preparation_path(id: lecture.id), - class: 'action-button' - %> - <% elsif joinable_time.past? && !lecture.attended? %> - <% start_label = if lecture.is_live? then - "Join" - else - "Start" - end %> - <%= button_to start_label, - lectures_start_path(id: lecture.id), - class: 'action-button' - %> - <% else %> -
- <%= button_to "Review", - lecture_start_review_path(id: lecture.id), - class: 'action-button' - %> -
- <% end %> - +
+ <%= render partial: 'lectures/lecture_action', locals: { lecture: } %> +
+ +
+ <% if lecture.recording %> + <%= link_to "Open recording", lecture.recording&.recording_url %> + <% end %> +
+
- - <% if lecture.recording %> - <%= link_to "Open recording", lecture.recording&.recording_url %> - <% end %> - - -<% end %>