Skip to content
This repository was archived by the owner on Apr 27, 2026. It is now read-only.

Make laptop specific packages optional#1119

Merged
Jas-SinghFSU merged 12 commits into
Jas-SinghFSU:masterfrom
bitwise74:master
Jan 3, 2026
Merged

Make laptop specific packages optional#1119
Jas-SinghFSU merged 12 commits into
Jas-SinghFSU:masterfrom
bitwise74:master

Conversation

@bitwise74

Copy link
Copy Markdown
Contributor

This pull request is directly related to issue #1116. It makes the bluez and bluez-utils packages optional and improves feedback for when a user wants to use these bar components with missing packages

New bluetooth component:
image
The label now explicitly states "Unavailable" instead of just saying "Off" and clicking on the element lets the user know the bluetooth service is missing

New power menu:
image
In case brightnessctl is missing, the slider gets replaced with a message saying it wasn't found on the machine

Comment thread src/components/menus/energy/brightness/index.tsx Outdated
@Jas-SinghFSU

Copy link
Copy Markdown
Owner

Can we add this in our style/scss/common/common.scss. I thought i had a generic dim style but I guess not:

.dim {
    opacity: 0.5;
}

Comment thread src/core/initialization/index.ts Outdated
@Jas-SinghFSU

Copy link
Copy Markdown
Owner

Seems like there are some linting errors. Could you run npm run lint:fix?

@bitwise74

Copy link
Copy Markdown
Contributor Author

Oops looks like my vscode config removed the semicolos and i didnt notice. I'm on it

Copilot AI review requested due to automatic review settings January 3, 2026 20:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Jas-SinghFSU Jas-SinghFSU merged commit 83423eb into Jas-SinghFSU:master Jan 3, 2026
1 check passed
@Jas-SinghFSU

Copy link
Copy Markdown
Owner

Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants