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

Merged
merged 2 commits into from
Feb 19, 2025

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
@EbonJaeger EbonJaeger force-pushed the fix/workspace-applet-wayland-support branch from d1bf679 to 6f9c937 Compare February 12, 2025 21:39
JoshStrobl and others added 2 commits February 18, 2025 16:46
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.
Signed-off-by: Evan Maddock <maddock.evan@vivaldi.net>
@EbonJaeger EbonJaeger force-pushed the fix/workspace-applet-wayland-support branch from 6f9c937 to 4c32aa2 Compare February 18, 2025 21:46
@JoshStrobl JoshStrobl changed the title Fix/workspace applet wayland support fix: workspace applet wayland support Feb 19, 2025
@JoshStrobl JoshStrobl self-requested a review February 19, 2025 12:12
Copy link
Contributor

@fossfreedom fossfreedom left a comment

Choose a reason for hiding this comment

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

As you say - key is that it fixes the crash. Its actually very usable. Lets get this in and iterate on it with later changes.

Nicely done!

@fossfreedom fossfreedom merged commit dd8bc35 into main Feb 19, 2025
1 check passed
@fossfreedom fossfreedom deleted the fix/workspace-applet-wayland-support branch February 19, 2025 20:38
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: Done
Development

Successfully merging this pull request may close these issues.

Implement Wayland support in Workspaces applet
3 participants