Skip to content

Conversation

@0xda157
Copy link
Contributor

@0xda157 0xda157 commented Jul 27, 2025

Submission Checklist

Notify Maintainers

@MrSom3body

@stylix-automation stylix-automation bot added topic: testbed Testbed changes topic: modules /modules/ subsystem labels Jul 27, 2025
@0xda157 0xda157 added the backport: release-25.05 Changes to release-25.05 stable branch label Jul 27, 2025
@MrSom3body
Copy link
Contributor

Is it okay to set a keybind for fuzzel? Otherwise, it’s hard to open the launcher again

Copy link
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.

Is it okay to set a keybind for fuzzel? Otherwise, it’s hard to open the launcher again

Good point. However, since this is the case with nearly all testbeds, it would be better to generally solve this issue by providing a consistent interface for re-running the test code inside the testbed. I suppose this involves installing stylix.testbed.ui.application and stylix.testbed.ui.command.text into the testbed. I will open a PR for this later.

In the meantime, the testbed should be fine.

@0xda157 0xda157 merged commit 683d626 into nix-community:master Jul 28, 2025
8 checks passed
@0xda157 0xda157 deleted the fuzzel-testbed branch July 28, 2025 16:16
stylix-automation bot pushed a commit that referenced this pull request Jul 28, 2025
Link: #1784

Reviewed-by: NAHO <[email protected]>
Reviewed-by: Karun Sandhu <[email protected]>
(cherry picked from commit 683d626)
@stylix-automation
Copy link
Contributor

Successfully created backport PR for release-25.05:

0xda157 added a commit that referenced this pull request Jul 28, 2025
Link: #1784

Reviewed-by: NAHO <[email protected]>
Reviewed-by: Karun Sandhu <[email protected]>
(cherry picked from commit 683d626)
@trueNAHO
Copy link
Member

Is it okay to set a keybind for fuzzel? Otherwise, it’s hard to open the launcher again

Good point. However, since this is the case with nearly all testbeds, it would be better to generally solve this issue by providing a consistent interface for re-running the test code inside the testbed. I suppose this involves installing stylix.testbed.ui.application and stylix.testbed.ui.command.text into the testbed.

This should be resolved by commit 662dd8a ("stylix/testbed/modules/application: allow re-running test code") once it gets merged in #1662.

make-42 pushed a commit to make-42/stylix that referenced this pull request Sep 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport: release-25.05 Changes to release-25.05 stable branch topic: modules /modules/ subsystem topic: testbed Testbed changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants