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

feat: add option to hide Submap module if there is no submap active#1081

Closed
ashton wants to merge 3 commits into
Jas-SinghFSU:masterfrom
ashton:master
Closed

feat: add option to hide Submap module if there is no submap active#1081
ashton wants to merge 3 commits into
Jas-SinghFSU:masterfrom
ashton:master

Conversation

@ashton

@ashton ashton commented Aug 9, 2025

Copy link
Copy Markdown

Adding an option to settings and hiding bar's submap module, if there is no submap active.

Implements #1076 alternative

@ashton ashton changed the title Hide Submap module if there is no submap active feat: add option to hide Submap module if there is no submap active Aug 9, 2025
@Jas-SinghFSU

Copy link
Copy Markdown
Owner

Great work! Could you please add the GUI config option to src/components/bar/settings/config.tsx:

                <Option
                    opt={options.bar.customModules.submap.hideOnDefault}
                    title="Hide On Default"
                    subtitle="Hides the SubMap module if it is on the default submap."
                    type="boolean"
                />

@Jas-SinghFSU

Copy link
Copy Markdown
Owner

And can you run npm run lint:fix to resolve the linter issues as well please 😄

lika85456 added a commit to lika85456/HyprPanel that referenced this pull request Feb 3, 2026
@Jas-SinghFSU

Copy link
Copy Markdown
Owner

Hey @ashton, thanks for this PR and for the time you put into it.

HyprPanel is being archived in the next couple of weeks in favor of Wayle (context: #1193). Closing this PR since there won't be any further HyprPanel releases. Your branch is unaffected, so anyone forking HyprPanel can pick this up.

If this also applies to Wayle, a PR at https://github.com/wayle-rs/wayle would be very welcome.

Thanks again! <3

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.

2 participants