Refine calendar and CI display flow with the Signal visual style - #23
Merged
Merged
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 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. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Urgent calendar screens previously squeezed the event title into 19 pixels or removed it in the final minute; CI status could scroll out of view with its details. This refresh gives all three integrations a consistent Signal visual style while making those states easier to read.
IN <1min the final minute, and drain the active-event track against its duration.CI FAIL/CI WAIT/CI OKfixed above scrolling context; move the running spinner below the title and label both quota readouts.Event selection, priorities, polling, overlay rotation/dwell, timeouts, quiet hours, and manual-session precedence retain their existing contracts. No new dependencies, host animation loop, configuration, or service. This PR has not been activated in production.
Full before/after captures and animation preview
Validation: 445 tests pass, wheel and source distribution build, and
git diff --checkpasses. Firmware 1.2.3 / API 27.5.0 framebuffer captures cover the calendar and CI states, including<1m,ALL CLEAR, andREST LEFT. The encoded Nyan animation was uploaded under a separate preview app, read back byte for byte, and successfully drawn. Preview frames expired after five seconds and used application-scoped cleanup. The 24 source PNGs and encoded asset independently reproduce byte for byte.Design review: full-depth (Pro) plan cross-check for the multi-integration presentation change; two logical Gemini advisory rounds converged after geometry corrections. The native model identity was not attested, so this is advisory evidence rather than an attested independent-family gate. An Astra high review of the integrated source and device captures found no blocking issues; physical captures caught and corrected an overlong healthy-state label before this commit.