Skip to content

Conversation

@Kiara65
Copy link
Contributor

@Kiara65 Kiara65 commented Feb 5, 2026

Summary

Fixed z-index stacking issue where quiz page elements were overlaying dropdowns in the course title bar, making them inaccessible to users.

Checklist

General

Client

  • I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

This PR fixes an issue where the quiz title/submission bar overlaps and partially hides the notifications popover, preventing users from seeing it completely.
Solves #11692

Description

  • Increased z-index of #course-title-bar-test to 1030 to ensure dropdowns are displayed above quiz page content

Steps for Testing

Prerequisites:

  • 1 Instructor
  1. Log in to Artemis
  2. Navigate to Course Management
  3. Go to any course and open its exercises
  4. Create a new quiz exercise
  5. In the exercise overview, navigate to the new quiz and start it
Screenshot 2026-02-05 at 09 29 02
  1. Switch to the student view and start the quiz
  2. Click the eye and bell icons in the quiz bar and verify that the dropdowns are displayed correctly and are not hidden

Testserver States

You can manage test servers using Helios. Check environment statuses in the environment list. To deploy to a test server, go to the CI/CD page, find your PR or branch, and trigger the deployment.

Review Progress

Code Review

  • Code Review

Manual Tests

  • Test

Screenshots

Before:
Screenshot 2026-02-05 at 09 39 49

Fixed:
Screenshot 2026-02-05 at 09 35 56

Summary by CodeRabbit

  • Style
    • Adjusted z-index layering for improved visual stacking of course title bar interface elements.

@Kiara65 Kiara65 linked an issue Feb 5, 2026 that may be closed by this pull request
@github-project-automation github-project-automation bot moved this to Work In Progress in Artemis Development Feb 5, 2026
@github-actions github-actions bot added client Pull requests that update TypeScript code. (Added Automatically!) core Pull requests that affect the corresponding module labels Feb 5, 2026
@helios-aet helios-aet bot temporarily deployed to artemis-test1.artemis.cit.tum.de February 5, 2026 08:41 Inactive
@Kiara65 Kiara65 marked this pull request as ready for review February 5, 2026 08:46
@Kiara65 Kiara65 requested a review from krusche as a code owner February 5, 2026 08:46
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

End-to-End (E2E) Test Results Summary

Test Strategy: Two-phase execution

  • Phase 1 (Relevant): e2e/Login.spec.ts e2e/Logout.spec.ts e2e/SystemHealth.spec.ts e2e/course/CourseManagement.spec.ts
  • Phase 2 (Remaining): e2e/atlas/ e2e/exam/ e2e/exercise/ e2e/lecture/

Status: All E2E tests passed (both phases)

TestsPassed ✅SkippedFailedTime ⏱
End-to-End (E2E) Test Report23 ran23 passed0 skipped0 failed4m 35s 943ms
TestResultTime ⏱
No test annotations available

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

A new CSS class selector .course-title-bar-test with z-index: 1030; was added to the course title bar component stylesheet. No other logic or styling was modified.

Changes

Cohort / File(s) Summary
Course Title Bar Styling
src/main/webapp/app/core/course/shared/course-title-bar/course-title-bar.component.scss
Added new CSS class .course-title-bar-test with z-index: 1030; rule.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary fix: resolving an issue where quiz bar elements were hiding notifications. This directly corresponds to the z-index change made in the CSS file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bugfix/quiz-title-and-submission-bar-hides-notifications

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@helios-aet helios-aet bot temporarily deployed to artemis-test1.artemis.cit.tum.de February 5, 2026 10:59 Inactive
Copy link

@mariyakoeva mariyakoeva left a comment

Choose a reason for hiding this comment

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

LGTM, tested on TS1

Copy link

@Jess-hub09 Jess-hub09 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 on TS1 with Test instructor 17 (Google Chrome). Unfortunately the issue is still reproducible on my side. The dropdowns are still hidden when starting the quiz in student view.
Image

@helios-aet helios-aet bot temporarily deployed to artemis-test4.artemis.cit.tum.de February 5, 2026 21:41 Inactive
@helios-aet helios-aet bot temporarily deployed to artemis-test1.artemis.cit.tum.de February 6, 2026 07:55 Inactive
@Kiara65
Copy link
Contributor Author

Kiara65 commented Feb 6, 2026

I tested this on TS1 with Test instructor 17 (Google Chrome). Unfortunately the issue is still reproducible on my side. The dropdowns are still hidden when starting the quiz in student view. Image

I was not able to reproduce the error in Chrome. Could you inspect the page, go to elements, click on the div below the jhi-course-title-bar (as seen in the screenshot), and check if the z-index is shown? Otherwise, it could be that the deployment failed or you deployed the wrong branch.
Screenshot 2026-02-06 at 09 10 33

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Phase 1: E2E Tests ✅

Status: Phase 1 passed

Tests Run: e2e/Login.spec.ts e2e/Logout.spec.ts e2e/SystemHealth.spec.ts e2e/course/CourseManagement.spec.ts

Details: Check the Phase 1 Test Report for detailed results.


This is an automated comment for Phase 1 of the E2E test execution.

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Phase 2: E2E Tests ✅

Status: Phase 2 passed

Tests Run: e2e/atlas/ e2e/exam/ e2e/exercise/ e2e/lecture/

Details: Check the Phase 2 Test Report for detailed results.


This is an automated comment for Phase 2 of the E2E test execution.

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

End-to-End (E2E) Test Results Summary

Test Strategy: Two-phase execution

  • Phase 1 (Relevant): e2e/Login.spec.ts e2e/Logout.spec.ts e2e/SystemHealth.spec.ts e2e/course/CourseManagement.spec.ts
  • Phase 2 (Remaining): e2e/atlas/ e2e/exam/ e2e/exercise/ e2e/lecture/

Status: All E2E tests passed (both phases)

Detailed Results: Check the individual phase reports in the workflow run for test counts, timing, and results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client Pull requests that update TypeScript code. (Added Automatically!) core Pull requests that affect the corresponding module ready for review

Projects

Status: Work In Progress

Development

Successfully merging this pull request may close these issues.

Quiz Title/Submission Bar hides Notifications

3 participants