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

Multiple changes for Hyprland.ts #257

Merged
merged 6 commits into from
Jan 14, 2024
Merged

Multiple changes for Hyprland.ts #257

merged 6 commits into from
Jan 14, 2024

Conversation

Rykerh
Copy link
Contributor

@Rykerh Rykerh commented Jan 14, 2024

-added _syncClients call to moveworkspaces case so that clients show their correct position and monitor after a workspace is moved between monitors

-added _syncClients call to closewindow case so that client x,y coordinate and size values are updated when they fill the newly available screen space

-moved 'event' emit command to after the switch statement so that the 'Hyprland' service clients/workspaces/monitors are up-to-date by the time the user receives the signal

Rykerh and others added 6 commits January 13, 2024 23:24
fix for client data being incorrect whenever moving workspaces between monitors
ensure client size and x,y coordinate values are updated after closing a window
moved this.emit('event',e,params) to after the switch statement so that all relevant 'Hyprland' Service data is updated before signalling the user that an event has occured
@Aylur Aylur linked an issue Jan 14, 2024 that may be closed by this pull request
@Aylur Aylur merged commit 4939aea into Aylur:main Jan 14, 2024
2 checks passed
gorsbart pushed a commit to gorsbart/ags that referenced this pull request Feb 28, 2024
-added _syncClients call to moveworkspaces case so that clients show their correct position and monitor after a workspace is moved between monitors

-added _syncClients call to closewindow case so that client x,y coordinate and size values are updated when they fill the newly available screen space

-moved 'event' emit command to after the switch statement so that the 'Hyprland' service clients/workspaces/monitors are up-to-date by the time the user receives the signal
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.

Multi-monitor issue: Hyprland.workspaces and Hyprland.clients out of sync
2 participants