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.

Sign up for GitHub

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

Jump to bottom

Windows are disappearing from screen when they resized and placed again #338

Closed
NsCDE opened this issue Dec 9, 2020 · 1 comment · Fixed by #341
Closed

Windows are disappearing from screen when they resized and placed again #338

NsCDE opened this issue Dec 9, 2020 · 1 comment · Fixed by #341
Assignees
@ThomasAdam
Labels
type:bug Something's broken!
Milestone
1.0.2

Comments

@NsCDE
Copy link
Contributor

NsCDE commented Dec 9, 2020

Hi Thomas.

This is reproducable on multiple, as well as on single monitor setup. Master branch.

DestroyFunc f_ZoomInResize
AddToFunc f_ZoomInResize

  • I ThisWindow (CirculateHit, Maximized) Maximize false
  • I ThisWindow (CirculateHit, Maximized) State 8 false
  • I ResizeMove w+50p w+50p w-25p w-25p ewmhiwa
  • I ThisWindow (CirculateHit, !Iconic) PlaceAgain

I have isolated that pure "PlaceAgain" is the one which is removing window from the screen.

During invisibility, FvwmPager still shows disapeared window in it's place.

Workaround is to go to some other desk and return back, then window will be there again.
@ThomasAdam ThomasAdam added the type:bug Something's broken! label Dec 9, 2020
@ThomasAdam ThomasAdam self-assigned this Dec 9, 2020
@ThomasAdam ThomasAdam added this to the 1.0.2 milestone Dec 9, 2020
ThomasAdam added a commit that referenced this issue Dec 9, 2020
@ThomasAdam
PlaceAgain: don't clobber desk
397894e 
When a window is being asked to PlaceAgain, the window was being forced
to the wrong desk due to the style flags assuming the monitor was
relative to itself.

This doesn't need to happen for windows being asked to PlaceAgain.

Fixes #338
@ThomasAdam ThomasAdam mentioned this issue Dec 9, 2020
Merged
@NsCDE
Copy link
Contributor Author

NsCDE commented Dec 10, 2020

This fixed the issue definitely.

@NsCDE NsCDE closed this as completed Dec 10, 2020
ThomasAdam added a commit that referenced this issue Dec 10, 2020
@ThomasAdam
PlaceAgain: don't clobber desk
c7372cf 
When a window is being asked to PlaceAgain, the window was being forced
to the wrong desk due to the style flags assuming the monitor was
relative to itself.

This doesn't need to happen for windows being asked to PlaceAgain.

Fixes #338
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThomasAdam ThomasAdam

Labels
type:bug Something's broken!
Projects
FVWM3
Status: Done
Milestone
1.0.2
Development

Successfully merging a pull request may close this issue.

PlaceAgain: don't clobber desk fvwmorg/fvwm3
2 participants
@ThomasAdam @NsCDE