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

IconMan is in the wrong spot #879

Closed
polarbub opened this issue Aug 17, 2023 · 2 comments · Fixed by #884
Closed

IconMan is in the wrong spot #879

polarbub opened this issue Aug 17, 2023 · 2 comments · Fixed by #884
Assignees
Labels
type:bug Something's broken!
Milestone

Comments

@polarbub
Copy link

Upfront Information

  • Fvwm3 version: fvwm3 1.0.7 (released)

  • Distro: Arch Linux

  • Platform: Linux unknown

Expected Behaviour

When starting fvwm3 the IconMan should be on the right side of the primary monitor.

Actual Behaviour

The IconMan was on the right side of the whole X screen on my second monitor. It seems like fvwm knows that it should be on my left monitor because it won't maximize windows over where it should be. It's trivial to move it as shown in the video, it is just a little bit annoying.

Steps to Reproduce

  1. Connect two monitors
  2. Set left monitor as primary via randr
  3. Start fvwm3 with the default configuration
  4. Observe the IconMan being in the wrong place

Extra info

This doesn't happen on version 1.0.6a

out.mov
@polarbub polarbub added the type:bug Something's broken! label Aug 17, 2023
@ThomasAdam ThomasAdam added this to the 1.0.8 milestone Aug 18, 2023
@ThomasAdam ThomasAdam self-assigned this Aug 18, 2023
@ThomasAdam
Copy link
Member

Does this fix it?

diff --git a/default-config/config b/default-config/config
index 0d4da94b7..203bd27e7 100644
--- a/default-config/config
+++ b/default-config/config
@@ -613,7 +613,7 @@ DestroyModuleConfig FvwmScript:*
 #
 # Note - To use the system tray you must have "stalonetray" installed.
 DestroyModuleConfig RightPanel:*
-*RightPanel: Geometry 120x$[monitor.$[monitor.primary].height]-0+0
+*RightPanel: Geometry 120x$[monitor.$[monitor.primary].height]-0+0@p
 *RightPanel: Colorset 10
 *RightPanel: Rows $[monitor.$[monitor.primary].height]
 *RightPanel: Columns 120

@polarbub
Copy link
Author

Yep! Works like a charm!

@ThomasAdam ThomasAdam reopened this Aug 21, 2023
ThomasAdam added a commit that referenced this issue Aug 26, 2023
When setting up the geometry of the RightPanel, explicitly state that
the geometry is intended to be used on the primary monitor.

Fixes #879
ThomasAdam added a commit that referenced this issue Aug 26, 2023
When setting up the geometry of the RightPanel, explicitly state that
the geometry is intended to be used on the primary monitor.

Fixes #879
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something's broken!
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants