Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Hyprland's special workspaces aren't cleared #2540

Merged
merged 2 commits into from Sep 25, 2023

Conversation

Syndelis
Copy link
Contributor

@Syndelis Syndelis commented Sep 25, 2023

Fixes #2505

About this PR

This PR makes sure special workspaces can be removed. Before this, the code for Workspaces::remove_workspace() didn't account for the workspaces starting with special: in their names, which resulted in the workspace not being found and, thus, not being removed.

Additional changes

During the investigation of this bug, I also stumbled upon Hyprland creating special workspaces twice. The second workspace always seems to be called special: + the name of the first special workspace, resulting in "doubled specials" (special:special:<some_name>). This PR also prevents the creation (and deletion) of workspaces named special:special:, which served no purpose and would linger a few seconds in the user's bar. See hyprwm/Hyprland#3424 for more info.

Related PR

Syndelis and others added 2 commits September 25, 2023 17:12
fixes  Alexays#2505

Co-authored-by: Gabriel Fox <Inbox@GabrielFox.Dev>
see hyprwm/Hyprland#3424 for more info

Co-authored-by: Gabriel Fox <Inbox@GabrielFox.Dev>
@Alexays Alexays merged commit 4c0347d into Alexays:master Sep 25, 2023
@Syndelis Syndelis deleted the fix/hyprland-special-workspaces branch September 25, 2023 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hyprland/Workspaces: icon for named special workspace is not cleared on toggle
2 participants