Skip to content

[Feature:InstructorUI] Image Annotator Access via Filename Click#12207

Closed
Sourojitbanerjee wants to merge 2 commits intoSubmitty:mainfrom
Sourojitbanerjee:feature/image-filename-annotator-click
Closed

[Feature:InstructorUI] Image Annotator Access via Filename Click#12207
Sourojitbanerjee wants to merge 2 commits intoSubmitty:mainfrom
Sourojitbanerjee:feature/image-filename-annotator-click

Conversation

@Sourojitbanerjee
Copy link
Copy Markdown
Contributor

@Sourojitbanerjee Sourojitbanerjee commented Nov 11, 2025

This PR closes #12000

Why is this Change Important & Necessary?

Allow clicking image filenames to open the annotator directly, improving usability by providing a larger click target alongside the existing arrow button.

What is the New Behavior?

  • Image files now open the annotator directly when clicking the filename
  • Non-image files maintain the original behavior
  • Provides larger click target for better usability while keeping the existing arrow button functionality

What steps should a reviewer take to reproduce or test the bug or new feature?

  1. Navigate to a submission with image files
  2. Click on image filenames - should open annotator directly
  3. Click on non-image filenames - should maintain original behavior
  4. Verify both filename click and arrow button work correctly

Automated Testing & Documentation

  • This feature needs automated tests (create issue if not present)
  • Documentation should be updated on submitty.org (create issue if needed)

Other information

  • This is not a breaking change
  • No migrations required
  • No security concerns identified

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.71%. Comparing base (58b54df) to head (7bce2d9).
⚠️ Report is 29 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #12207   +/-   ##
=========================================
  Coverage     21.71%   21.71%           
  Complexity     9612     9612           
=========================================
  Files           268      268           
  Lines         36073    36073           
  Branches        475      475           
=========================================
  Hits           7832     7832           
  Misses        27770    27770           
  Partials        471      471           
Flag Coverage Δ
autograder 21.39% <ø> (ø)
js 2.07% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.71% <ø> (ø)
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
Copy Markdown
Contributor

@RyanStyron RyanStyron left a comment

Choose a reason for hiding this comment

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

The feature works as intended. Thank you for the contribution. Please resolve the two failing checks as well as merge conflicts before approval.

@github-project-automation github-project-automation bot moved this from Seeking Reviewer to Work in Progress in Submitty Development Nov 14, 2025
@Sourojitbanerjee
Copy link
Copy Markdown
Contributor Author

Thank you! I've resolved the merge conflicts. Could you please approve the workflows and let me know if there are any specific CSS lint errors I need to fix? The feature is working as intended.

Copy link
Copy Markdown
Contributor

@RyanStyron RyanStyron left a comment

Choose a reason for hiding this comment

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

This feature no longer works as popOutSubmittedFile was replaced by openSubmittedFile in #12135.

@github-actions github-actions bot added the Abandoned PR - Needs New Owner No activity on PR for more than 2 weeks -- seeking new owner to complete label Dec 1, 2025
@RyanStyron RyanStyron changed the title Enable image annotator access by clicking filename [Enhancement:InstructorUI] Image Annotator Access via Filename Click Dec 15, 2025
@RyanStyron RyanStyron changed the title [Enhancement:InstructorUI] Image Annotator Access via Filename Click [Feature:InstructorUI] Image Annotator Access via Filename Click Dec 15, 2025
@RyanStyron RyanStyron closed this Dec 15, 2025
@github-project-automation github-project-automation bot moved this from Work in Progress to Done in Submitty Development Dec 15, 2025
bmcutler pushed a commit that referenced this pull request Dec 15, 2025
### Why is this Change Important & Necessary?
Closes #12000; supersedes
#12207.
As noted in the linked issue, clicking submission images to open the
annotator directly is not currently a feature. Enabling the filename to
be clicked to open the image/annotator better aligns usability with
other submission types.

### What is the New Behavior?
Clicking an image filename as requested by the linked issue is now
functional.

### What steps should a reviewer take to reproduce or test the bug or
new feature?
(_Copied from superseded PR_)
1. Navigate to a submission with image files
2. Click on image filenames - should open annotator directly
3. Click on non-image filenames - should maintain original behavior
4. Verify both filename click and arrow button work correctly

---------

Co-authored-by: Sourojit Banerjee <181811476+Sourojitbanerjee@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Abandoned PR - Needs New Owner No activity on PR for more than 2 weeks -- seeking new owner to complete

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Make Annotatable Image File Names Clickable in Grader View

2 participants