Skip to content

updated system_config.md#2426

Merged
openminddev merged 3 commits intomainfrom
screen-management-docs
Mar 7, 2026
Merged

updated system_config.md#2426
openminddev merged 3 commits intomainfrom
screen-management-docs

Conversation

@Prachi1615
Copy link
Copy Markdown
Collaborator

Overview

This PR updates the docs for setting screen animation

Copilot AI review requested due to automatic review settings February 26, 2026 20:40
@Prachi1615 Prachi1615 requested review from a team as code owners February 26, 2026 20:40
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Feb 26, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the documentation for the Screen Animation Service configuration in the BrainPack full autonomy guidelines. The changes improve the reliability of the fullscreen kiosk mode by using wmctrl to force fullscreen behavior after Chromium launches, addressing potential issues with the --kiosk flag alone.

Changes:

  • Added wmctrl as a required dependency alongside unclutter
  • Modified the startup script to run Chromium in the background and use wmctrl to enforce fullscreen mode
  • Added process lifecycle management with PID tracking and wait to properly maintain the systemd service

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@openminddev openminddev merged commit 9ffd94b into main Mar 7, 2026
8 checks passed
@openminddev openminddev deleted the screen-management-docs branch March 7, 2026 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants