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

xdotool can't move windows #25

Closed
morgant opened this issue Aug 21, 2021 · 2 comments
Closed

xdotool can't move windows #25

morgant opened this issue Aug 21, 2021 · 2 comments
Assignees
Milestone

Comments

@morgant
Copy link
Owner

morgant commented Aug 21, 2021

When using xdotool under OpenBSD 6.9, xdotool's windowsize command works to resize windows, but windowmove does not result in a window being moved. If I switch to fvwm, both windowsize & windowmove work correctly.

@morgant morgant changed the title xdotool can' xdotool can't move windows Aug 21, 2021
@morgant
Copy link
Owner Author

morgant commented Sep 3, 2021

It seems that the code in event.c's handle_configure_request() which would set tmp_win->frame_x & tmp_win->frame_y is commented out. Uncommenting it results in xdotool being able to successfully move MLVWM windows using its windowmove command.

It seems to prevent the window from being placed off the left or top of the screen, so I don't know if it needs to be smarter or not.

morgant added a commit that referenced this issue Sep 5, 2021
…gure_request() to fix xdotool not moving windows. Issue #25
morgant added a commit that referenced this issue Sep 5, 2021
…g SetUpFrame(), plus reduced vars used. Issue #25
@morgant
Copy link
Owner Author

morgant commented Sep 5, 2021

With that code block uncommented, I did end up removing the constraint preventing windows from being moved off screen to the left, but kept the constraint preventing them from being moved above the menu bar. I also cleaned up the code a little bit to not modify tmp_win frame values before passing to SetUpFrame(), which reduced the number of variables needed, as well.

@morgant morgant self-assigned this Sep 5, 2021
@morgant morgant added this to the 0.9.4 milestone Sep 5, 2021
@morgant morgant closed this as completed Sep 5, 2021
morgant added a commit that referenced this issue Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant