diff --git a/Changelog.md b/Changelog.md
index 134ee919bd..957cd8f1c8 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -6,6 +6,7 @@
### ✨ New features and improvements
- Improved layout and labeling in the assignment settings form for both standard and timed assessments. (#7531)
+- Improved Assignment view for students (#7533)
### 🐛 Bug fixes
diff --git a/app/views/assignments/_read.html.erb b/app/views/assignments/_read.html.erb
index 7037e4f12f..d8cf2098b9 100644
--- a/app/views/assignments/_read.html.erb
+++ b/app/views/assignments/_read.html.erb
@@ -1,8 +1,4 @@
-<% short_identifier = @assignment.is_peer_review? ?
- "#{@assignment.parent_assignment.short_identifier} #{PeerReview.model_name.human}" :
- @assignment.short_identifier %>
-
-
<%= "#{short_identifier}: #{@assignment.description}" %>
+<%= t('assignments.assignment_information') %>
<% if @assignment.message %>
@@ -134,6 +130,32 @@
<% end %>
<% end %>
+ <% unless @assignment.starter_file_updated_at.nil? || @grouping.nil? %>
+ <% if allowed_to?(:download_starter_file?, @grouping) %>
+ <%= I18n.t('assignments.starter_file.title').downcase.capitalize %>
+ <%= I18n.t('assignments.starter_file.provided') %>
+ <%= I18n.t('assignments.starter_file.changed_at',
+ changed_date: I18n.l(@assignment.starter_file_updated_at)) %>
+
+ <%= button_to t(:download_the, item: t('assignments.starter_file.title')),
+ download_starter_file_course_assignment_groups_path(@current_course, @assignment),
+ method: 'get' %>
+
+ <% # Display the URL of this group's repository if applicable
+ if allowed_to?(:access_repo?, @grouping) %>
+
+ <%= button_to t('assignments.starter_file.populate_repo'),
+ populate_repo_with_starter_files_course_assignment_groups_path(
+ course_id: @assignment.course_id,
+ assignment_id: @assignment.id
+ ),
+ { method: :patch,
+ data: { confirm: t('assignments.starter_file.populate_repo_confirm'),
+ disable_with: t(:please_wait) } } %>
+
+ <% end %>
+ <% end %>
+ <% end %>
<% if !peer_review && (!@assignment.is_timed || !@grouping&.start_time.nil? || @grouping&.past_collection_date?) %>
<%= Submission.model_name.human.pluralize %>
<% if @grouping.nil? %>
@@ -146,19 +168,20 @@
<% end %>
+ - <%= link_to t('submissions.student.files_submitted', count: @num_submitted_files),
+ file_manager_course_assignment_submissions_path(course_id: @assignment.course_id,
+ assignment_id: @assignment.id) %>
+
<% if @num_submitted_files > 0 %>
-
<%= t('submissions.student.last_revision_date') %>
<%= l(@last_modified_date) %>
<% end %>
- - <%= t('submissions.student.files_submitted', count: @num_submitted_files) %>
- <%= Assignment.human_attribute_name(:assignment_files) %>
- <% if @assignment.assignment_files.length == 0 %>
- <%= t('assignments.no_required_files') %>
- <% else %>
+ <% if @assignment.assignment_files.length > 0 %>
+ <%= Assignment.human_attribute_name(:assignment_files) %>
<% @assignment.assignment_files.each do |assignment_file| %>
- <%= assignment_file.filename %>
@@ -175,30 +198,4 @@
<% end %>
<% end %>
<% end %>
-<% unless @assignment.starter_file_updated_at.nil? || @grouping.nil? %>
- <% if allowed_to?(:download_starter_file?, @grouping) %>
- <%= I18n.t('assignments.starter_file.title') %>
- <%= I18n.t('assignments.starter_file.provided') %>
- <%= I18n.t('assignments.starter_file.changed_at',
- changed_date: I18n.l(@assignment.starter_file_updated_at)) %>
-
- <%= button_to t(:download_the, item: t('assignments.starter_file.title')),
- download_starter_file_course_assignment_groups_path(@current_course, @assignment),
- method: 'get' %>
-
- <% # Display the URL of this group's repository if applicable
- if allowed_to?(:access_repo?, @grouping) %>
-
- <%= button_to t('assignments.starter_file.populate_repo'),
- populate_repo_with_starter_files_course_assignment_groups_path(
- course_id: @assignment.course_id,
- assignment_id: @assignment.id
- ),
- { method: :patch,
- data: { confirm: t('assignments.starter_file.populate_repo_confirm'),
- disable_with: t(:please_wait) } } %>
-
- <% end %>
- <% end %>
-<% end %>
diff --git a/app/views/assignments/show.html.erb b/app/views/assignments/show.html.erb
index 8c2a692e3a..245abd3132 100644
--- a/app/views/assignments/show.html.erb
+++ b/app/views/assignments/show.html.erb
@@ -24,7 +24,11 @@
<% end %>
<% end %>
-<% content_for :title, "#{Assignment.model_name.human} #{@assignment.short_identifier}" %>
+<% short_identifier = @assignment.is_peer_review? ?
+ "#{@assignment.parent_assignment.short_identifier} #{PeerReview.model_name.human}" :
+ @assignment.short_identifier %>
+
+<% content_for :title, "#{short_identifier}: #{@assignment.description}" %>
diff --git a/config/locales/views/assignments/en.yml b/config/locales/views/assignments/en.yml
index cba1c405fd..8efdcd0134 100644
--- a/config/locales/views/assignments/en.yml
+++ b/config/locales/views/assignments/en.yml
@@ -2,6 +2,7 @@
en:
assignments:
assignment_has_groupings: Assignment has groupings.
+ assignment_information: Assignment information
average_annotations: "%{average_annotations} annotations per marked submission"
configuration_zip_file: Configuration Zip File
deadline_with_extension: You have an extension until %{extension_deadline}.
@@ -31,7 +32,6 @@ en:
hidden: "%{assignment_text} (hidden)"
manage_course_work: Manage Course Work
marking_scheme: 'Marking Scheme: %{identifier}'
- no_required_files: There are no required files for this assignment.
none: There are currently no assignments.
scanned_exam:
under_review: This exam is still under review.
@@ -64,7 +64,7 @@ en:
populate_repo_confirm: Are you sure you want to add the starter files to your repository? If they already exist in the repository, any changes will be overwritten.
populate_repo_error: Unfortunately, the starter files could not be added to your repository. Please download them as a zip file instead.
populate_repo_success: Starter files successfully added to your repository.
- provided: Starter files have been provided for this assignment
+ provided: Starter files have been provided for this assignment.
rename: Rename Selected File or Directory
starter_file_rule: Starter File Assignment Rules
starter_file_rule_types:
diff --git a/config/locales/views/groups/en.yml b/config/locales/views/groups/en.yml
index d7555b3b8b..5240f51fac 100644
--- a/config/locales/views/groups/en.yml
+++ b/config/locales/views/groups/en.yml
@@ -86,7 +86,7 @@ en:
not_allowed_to_delete_group: If you would like to remove this group, you must contact your instructor.
not_allowed_to_form_group: You are not allowed to form a group yourself. Please wait until your instructor forms your group.
section_groups_only: You can only form a group with students in your section.
- students_work_alone: Students work individually.
+ students_work_alone: You must complete this assignment individually.
url_repository: Repository URL
work_alone: Work alone
working_alone: You have indicated you are working individually on this assignment. If you change your mind and want to create or join a group, first delete your group using the button below.