Skip to content

fix(calendar): render now-line in front of task tiles#992

Merged
renemadsen merged 1 commit into
stablefrom
feat/calendar-now-line-front
Jun 9, 2026
Merged

fix(calendar): render now-line in front of task tiles#992
renemadsen merged 1 commit into
stablefrom
feat/calendar-now-line-front

Conversation

@renemadsen

Copy link
Copy Markdown
Member

Feedback

"Vis den aktuelle tidslinje i kalender (den røde streg) foran opgaverne og ikke bagved." — show the current-time line in front of the tasks, not behind.

Fix

The .now-line had z-index: 10, the same as task tiles (z 10–13, or 999 when raised), so tiles painted over it. Bumped .now-line to z-index: 1000 (above raised tiles) in both calendar-week-grid and the parallel calendar-day-column. It already has pointer-events: none, so it doesn't block clicks/drag on the tiles underneath. No e2e test references the now-line.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 9, 2026 05:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adjusts calendar styling so the “now” indicator (red current-time line) renders above task tiles in both week and day views, matching the reported UX requirement in the plugin’s Angular calendar module.

Changes:

  • Increased .now-line stacking order to ensure it paints in front of task tiles (including “raised” tiles).
  • Applied the same z-index adjustment consistently in both week-grid and day-column implementations.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
eform-client/src/app/plugins/modules/backend-configuration-pn/modules/calendar/components/calendar-week-grid/calendar-week-grid.component.scss Raises .now-line z-index so the current-time line renders above week-view task tiles.
eform-client/src/app/plugins/modules/backend-configuration-pn/modules/calendar/components/calendar-day-column/calendar-day-column.component.scss Mirrors the .now-line z-index change for the day-view column layout.

@renemadsen renemadsen merged commit b42880a into stable Jun 9, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants