Skip to content

enable graphical boot, bump ghafpkgs, adjust cosmic config#1339

Merged
brianmcgillion merged 1 commit intotiiuae:mainfrom
kajusnau:fix-brightness-on-boot
Sep 2, 2025
Merged

enable graphical boot, bump ghafpkgs, adjust cosmic config#1339
brianmcgillion merged 1 commit intotiiuae:mainfrom
kajusnau:fix-brightness-on-boot

Conversation

@kajusnau
Copy link
Copy Markdown
Collaborator

@kajusnau kajusnau commented Aug 27, 2025

Description of Changes

  1. Enabled graphical boot:
    • Splash screen with Ghaf logo will be displayed at boot
    • updated ghafpkgs to latest available to include ghaf logo
    • expanded existing ghaf graphics boot module with additional nix options
  2. Minor simplification of cosmic config via cosmic osd:
    • Cosmic already handles playing a ding sound on volume change via keyboard shortcut.
      This change removes our own workaround for that in favor of using Cosmic's own implementation

Type of Change

  • New Feature
  • Bug Fix
  • Improvement / Refactor

Related Issues / Tickets

Checklist

  • Clear summary in PR description
  • Detailed and meaningful commit message(s)
  • Commits are logically organized and squashed if appropriate
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • Author has run make-checks and it passes
  • All automatic GitHub Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing Instructions

Applicable Targets

  • Orin AGX aarch64
  • Orin NX aarch64
  • Lenovo X1 x86_64
  • Dell Latitude x86_64
  • System 76 x86_64

Installation Method

  • Requires full re-installation
  • Can be updated with nixos-rebuild ... switch
  • Other:

Test Steps To Verify:

  1. Boot laptop
  2. Verify graphical boot is enabled - splash screen with ghaf logo and loading animation is shown
  3. Verify boot continues to Cosmic greeter login
  4. Login
  5. Verify adjusting system volume with keyboard shortcuts (F2-F3) causes a sound to play

@kajusnau kajusnau force-pushed the fix-brightness-on-boot branch from 892ae3c to 56b0355 Compare August 27, 2025 13:22
@kajusnau kajusnau force-pushed the fix-brightness-on-boot branch from 56b0355 to b137597 Compare August 28, 2025 13:33
@kajusnau kajusnau changed the title desktop: fix low brightness at boot, adjust cosmic config enable graphical boot, bump ghafpkgs, adjust cosmic config Aug 28, 2025
@kajusnau kajusnau self-assigned this Aug 28, 2025
@kajusnau kajusnau force-pushed the fix-brightness-on-boot branch from b137597 to e90136f Compare August 29, 2025 07:38
@kajusnau kajusnau force-pushed the fix-brightness-on-boot branch from e90136f to 6985c5b Compare August 29, 2025 12:13
@kajusnau kajusnau force-pushed the fix-brightness-on-boot branch from 6985c5b to 5a687b7 Compare August 29, 2025 12:31
@kajusnau kajusnau force-pushed the fix-brightness-on-boot branch from 5a687b7 to bcdd3ed Compare August 29, 2025 12:55
@kajusnau kajusnau force-pushed the fix-brightness-on-boot branch from bcdd3ed to f95db56 Compare September 1, 2025 06:33
@kajusnau kajusnau marked this pull request as ready for review September 1, 2025 06:33
@kajusnau kajusnau added the Needs Testing CI Team to pre-verify label Sep 1, 2025
Copy link
Copy Markdown
Collaborator

@vunnyso vunnyso left a comment

Choose a reason for hiding this comment

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

Overall good, just minor comments. 🙂

@milva-unikie
Copy link
Copy Markdown

Tested on Lenovo-X1 (nixos-rebuild switch)

  • Ghaf logo is shown during boot
  • A sound is played when the volume is changed using keyboard shortcuts

Notes

  • The volume change sound is also played once right after login

@milva-unikie milva-unikie added Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon and removed Needs Testing CI Team to pre-verify labels Sep 2, 2025
- bump ghafpkgs
- enabled graphical boot with ghaf logo
- expanded existing ghaf graphics boot module
- minor simplification of cosmic config via cosmic osd

Signed-off-by: Kajus Naujokaitis <kajus.naujokaitis@unikie.com>
@kajusnau
Copy link
Copy Markdown
Collaborator Author

kajusnau commented Sep 2, 2025

Minor adjustments based on @vunnyso comments, no changes to functionality

Ready to merge

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 enables graphical boot with a Ghaf-branded splash screen and simplifies cosmic desktop configuration. The changes implement plymouth-based graphical boot with customizable logos and remove redundant volume change sound handling in favor of cosmic's native implementation.

Key changes:

  • Enable graphical boot with plymouth and Ghaf branding across different device profiles
  • Add cosmic-osd package overlay to fix audio playback compatibility
  • Simplify cosmic volume control configuration by leveraging built-in OSD functionality

Reviewed Changes

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

Show a summary per file
File Description
overlays/custom-packages/default.nix Adds cosmic-osd package to overlay
overlays/custom-packages/cosmic/cosmic-osd/default.nix Creates cosmic-osd override to use paplay instead of pw-play
modules/reference/profiles/mvp-user-trial.nix Removes commented boot graphics options and updates copyright
modules/reference/profiles/mvp-orinuser-trial.nix Explicitly disables graphical boot for Orin devices
modules/reference/profiles/mvp-orinuser-trial-extras.nix Explicitly disables graphical boot for Orin devices
modules/microvm/host/microvm-host.nix Enables graphical boot on microvm host
modules/desktop/graphics/cosmic/config/cosmic-config.nix Simplifies volume control shortcuts by removing custom sound playback
modules/desktop/graphics/boot.nix Significantly expands boot graphics module with new configuration options

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@brianmcgillion brianmcgillion merged commit 8e816d5 into tiiuae:main Sep 2, 2025
27 of 28 checks passed
@kajusnau kajusnau deleted the fix-brightness-on-boot branch October 7, 2025 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants