Skip to content

[Bugfix:TAGrading] Fix Stoplight Yellow#13023

Merged
bmcutler merged 1 commit into
mainfrom
fix-yellow-green-stoplight
Jul 10, 2026
Merged

[Bugfix:TAGrading] Fix Stoplight Yellow#13023
bmcutler merged 1 commit into
mainfrom
fix-yellow-green-stoplight

Conversation

@mattLif

@mattLif mattLif commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Why is this Change Important & Necessary?

On dark mode, "vibrant yellow" turns to a darker shade, causing the "limited access grader verified" stoplight to change color to not match the "Component graded by a limited access grader" stoplight. This PR fixes this by changing the former's yellow color to match the yellow used in the latter's css.

What is the New Behavior?

Before:
image

After
image

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

On dark mode, visit any gradeable's details page and notice the change in the legend

Automated Testing & Documentation

No automated testing

Other information

Not a breaking change
No migrations
Not a security risk

@github-project-automation github-project-automation Bot moved this to Seeking Reviewer in Submitty Development Jul 10, 2026
@mattLif mattLif changed the title Update server.css [Bugfix:TAGrading] Fix Stoplight Yellow Jul 10, 2026
@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.25%. Comparing base (717016a) to head (86d6eb1).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #13023   +/-   ##
=========================================
  Coverage     21.25%   21.25%           
  Complexity    10093    10093           
=========================================
  Files           278      278           
  Lines         37778    37778           
  Branches        516      516           
=========================================
  Hits           8031     8031           
  Misses        29244    29244           
  Partials        503      503           
Flag Coverage Δ
autograder 21.16% <ø> (ø)
js 1.98% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.16% <ø> (ø)
python_submitty_utils 79.83% <ø> (ø)
submitty_daemon_jobs 91.13% <ø> (ø)
vue 100.00% <ø> (ø)

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.

@dagemcn dagemcn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested, looks good, the grading page looks much better with this change.

@github-project-automation github-project-automation Bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Jul 10, 2026
@bmcutler
bmcutler merged commit 7fc5530 into main Jul 10, 2026
31 of 32 checks passed
@bmcutler
bmcutler deleted the fix-yellow-green-stoplight branch July 10, 2026 16:21
@github-project-automation github-project-automation Bot moved this from Awaiting Maintainer Review to Done in Submitty Development Jul 10, 2026
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.

3 participants