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/workspace applet wayland support #674

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

EbonJaeger
Copy link
Member

Description

This PR does a few things for the workspace applet:

  • Removes any reference to BudgieWM, since that is no longer used.
  • Removes any remaining Wnck/xid code.
  • Fixes a crash when a window is opened.
  • Makes the add workspace button unlickable if a new workspace cannot be added.

The only other outstanding issue that I know of so far is that hovering the mouse over the add workspace button when it's set to only show on hover causes the revealer to keep opening and closing in rapid succession for as long as the mouse is there.

This should fix #470. If any further refinements are needed, they can be done later in separate PRs.

Submitter Checklist

  • Squashed commits with git rebase -i (if needed)
  • Built budgie-desktop and verified that the patch worked (if needed)

@EbonJaeger EbonJaeger added the wayland Tasks related to port to Wayland label Feb 12, 2025
@EbonJaeger EbonJaeger added this to the 10.10 milestone Feb 12, 2025
@EbonJaeger EbonJaeger force-pushed the fix/workspace-applet-wayland-support branch from e0a304e to d1bf679 Compare February 12, 2025 21:37
@EbonJaeger EbonJaeger marked this pull request as ready for review February 12, 2025 21:37
JoshStrobl and others added 2 commits February 12, 2025 16:39
The big thing is that the applet no longer crashes when a new window is opened. When a window is opened, it may not yet have an application assigned to it. When that happens, set the item's icon to a generic one, and wait for it to update naturally. In practice, I have yet to see the generic icon; it's always been updated by the time I can see it.
@EbonJaeger EbonJaeger force-pushed the fix/workspace-applet-wayland-support branch from d1bf679 to 6f9c937 Compare February 12, 2025 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wayland Tasks related to port to Wayland
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Implement Wayland support in Workspaces applet
2 participants