Skip to content

feat(calendar): Fade passed events in calendar view - #4063

Open
borgsten wants to merge 1 commit into
noctalia-dev:mainfrom
borgsten:feature/faded_passed_events
Open

feat(calendar): Fade passed events in calendar view#4063
borgsten wants to merge 1 commit into
noctalia-dev:mainfrom
borgsten:feature/faded_passed_events

Conversation

@borgsten

@borgsten borgsten commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Draw already passed events in calendar view with reduced alpha to differentiate which has passed and which has not.

Motivation

Fading the passed events makes it easy on a glance to see what is upcoming and what has already passed.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Build / packaging

Testing

Added tests/calendar_view_passed_event_test.cpp to cover some edge cases.

Manual Coverage

  • Tested on Niri
  • Tested on Hyprland
  • Tested on Sway
  • Tested on another compositor:
  • Tested with different bar positions and density settings
  • Tested at different interface scaling values
  • Tested with multiple monitors

Screenshots / Videos

image

Checklist

  • This PR is ready for review, or it is marked as Draft.
  • I read and followed the relevant guidance in CONTRIBUTING.md.
  • I ran just format with clang-format v22+ installed, or this PR has no code changes.
  • I ran the relevant build or test commands, or explained why they were not run.
  • I self-reviewed the changes.
  • I checked for new warnings or errors.
  • I will update end-user documentation after merge, or this PR does not change user-facing configuration or behavior.
  • I added or updated assets/translations/en.json, or this PR adds no new user-facing strings.
  • I did not edit non-English translation files unless this PR is explicitly for translation tooling, an import/export sync, or a maintainer-requested locale change.
  • I used the existing canonical names for config keys, IPC names, paths, and identifiers.

Additional Notes

I added this as default behavior without any configuration option but if controlling this via config would be better then I can add that as well.

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.

1 participant