Skip to content

bottom: init#2181

Open
dastarruer wants to merge 3 commits intonix-community:masterfrom
dastarruer:add-bottom-module
Open

bottom: init#2181
dastarruer wants to merge 3 commits intonix-community:masterfrom
dastarruer:add-bottom-module

Conversation

@dastarruer
Copy link
Copy Markdown
Contributor

@dastarruer dastarruer commented Feb 1, 2026

Closes: #1834


Screenshots

image image

@stylix-automation stylix-automation bot added topic: home-manager Home Manager target topic: modules /modules/ subsystem topic: stylix /stylix/ subsystem labels Feb 1, 2026
Copy link
Copy Markdown
Member

@trueNAHO trueNAHO left a comment

Choose a reason for hiding this comment

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

Consider adding a testbed. They are extremely valuable.

@dastarruer dastarruer force-pushed the add-bottom-module branch 2 times, most recently from afe3eed to 6db7d36 Compare February 2, 2026 21:36
@dastarruer
Copy link
Copy Markdown
Contributor Author

dastarruer commented Feb 2, 2026

Consider adding a testbed. They are extremely valuable.

Yup, just did.

@dastarruer dastarruer requested a review from trueNAHO February 2, 2026 21:46
@trueNAHO trueNAHO changed the title bottom: init target bottom: init Feb 3, 2026
@stylix-automation stylix-automation bot added the topic: testbed Testbed changes label Feb 3, 2026
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is the selection background color in the nix run .#testbed:bottom:light testbed supposed to look like this:

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

definitely not...

Apologies, I added a testbed and then just tested dark mode, assuming that light mode would look fine. Will go back and fix this once I get the chance.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Looking at the configuration options, bottom currently does not support a global bg_color option. So far, if I set widgets.text.bg_color = base00, I get the following:

image

I have opened an issue here, and plan to submit a PR to add this feature.

@trueNAHO
Copy link
Copy Markdown
Member

Could you remove the seemingly unrelated and noisy commit 1b7bc78 ("treewide: deprecate manual targets.${target}.useWallpaper.enable options (#2084)")? If this is cherry-picked to resolve merge conflicts, just rebase on top of the master branch.

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

Labels

status: merge conflict Merge conflict topic: home-manager Home Manager target topic: modules /modules/ subsystem topic: nixos NixOS target topic: stylix /stylix/ subsystem topic: testbed Testbed changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bottom: add target

2 participants