Skip to content

[Feature:Submission] Open Submitted Files in New Tab#12135

Merged
bmcutler merged 10 commits intomainfrom
open-gradeable-files-in-new-tab
Nov 12, 2025
Merged

[Feature:Submission] Open Submitted Files in New Tab#12135
bmcutler merged 10 commits intomainfrom
open-gradeable-files-in-new-tab

Conversation

@RyanStyron
Copy link
Contributor

Why is this Change Important & Necessary?

This change is to simplify user experience in cases where there are several files submitted for a given Gradeable. When View is selected, the files now open in the same window as the one that is currently opened.

What is the New Behavior?

Prior to the change, an uploaded file which was submitted could be viewed by a student in the Gradeable and would appear as follows:
Screenshot 2025-10-10 165043
When View is clicked, a pop-up the contents of the file appear (in this case, the content of the file is text "Test file")
Screenshot 2025-10-10 165528

With this change:
Screenshot 2025-10-10 170744
image

@github-project-automation github-project-automation bot moved this to Seeking Reviewer in Submitty Development Oct 10, 2025
@automateprojectmangement automateprojectmangement bot moved this from Seeking Reviewer to In Review in Submitty Development Oct 10, 2025
@RyanStyron RyanStyron changed the title [Feature:Submission] Update File Pop-Up Functionality to Open Files in New Tab [Feature:Submission] Open Submitted Files in New Tab Oct 10, 2025
@codecov
Copy link

codecov bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.73%. Comparing base (fef0849) to head (165ec7c).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #12135   +/-   ##
=========================================
  Coverage     21.73%   21.73%           
  Complexity     9598     9598           
=========================================
  Files           268      268           
  Lines         36006    36006           
  Branches        475      475           
=========================================
  Hits           7827     7827           
  Misses        27708    27708           
  Partials        471      471           
Flag Coverage Δ
autograder 21.39% <ø> (ø)
js 2.07% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.74% <ø> (ø)
python_submitty_utils 80.08% <ø> (ø)
submitty_daemon_jobs 90.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@Rkoester47 Rkoester47 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this branch, and clicking "View" on a submitted and graded homework file did open it in a new tab. I think this is a good idea and much more intuitive than opening the file in a separate pop up window in front of the Submitty page. As far as I can tell the files always open in a new tab now, as intended.

@github-project-automation github-project-automation bot moved this from In Review to Awaiting Maintainer Review in Submitty Development Oct 14, 2025
Copy link
Contributor

@JManion32 JManion32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This request is based on our meeting where we determined that this change is only for students who are trying to view their submitted files.

In the TA grading panel, there was an option to popout files (see highlighted):
Screenshot 2025-10-14 174307
image

Now, it opens them on a separate page, which is consistent with the new functionality, but may be beyond the scope of what this PR intends to do.

@github-project-automation github-project-automation bot moved this from Awaiting Maintainer Review to Work in Progress in Submitty Development Oct 14, 2025
@RyanStyron
Copy link
Contributor Author

Updated changes to more accurately reflect intention of the PR.

@RyanStyron RyanStyron requested a review from JManion32 October 15, 2025 00:28
@automateprojectmangement automateprojectmangement bot moved this from Work in Progress to In Review in Submitty Development Oct 15, 2025
@github-project-automation github-project-automation bot moved this from In Review to Work in Progress in Submitty Development Oct 17, 2025
@RyanStyron RyanStyron requested a review from DarthNyan October 17, 2025 20:27
@automateprojectmangement automateprojectmangement bot moved this from Work in Progress to In Review in Submitty Development Oct 17, 2025
Copy link
Contributor

@DarthNyan DarthNyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes look good.

Copy link
Contributor

@JManion32 JManion32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes look good, this is a very positive change for student user experience!

@github-project-automation github-project-automation bot moved this from In Review to Awaiting Maintainer Review in Submitty Development Oct 20, 2025
@RyanStyron RyanStyron moved this from Awaiting Maintainer Review to Ready to Merge in Submitty Development Nov 11, 2025
@bmcutler bmcutler merged commit 3569768 into main Nov 12, 2025
25 checks passed
@bmcutler bmcutler deleted the open-gradeable-files-in-new-tab branch November 12, 2025 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants