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

Pager starts on leftmost monitor instead of primary one #880

Closed
dl8dtl opened this issue Aug 21, 2023 · 7 comments
Closed

Pager starts on leftmost monitor instead of primary one #880

dl8dtl opened this issue Aug 21, 2023 · 7 comments
Labels
skip:changelog Issue/PR should skip CHANGELOG type:bug Something's broken!

Comments

@dl8dtl
Copy link

dl8dtl commented Aug 21, 2023

Upfront Information

Please provide the following information by running the command and providing
the output.

  • Fvwm3 version (run: fvwm3 --version)

fvwm3 1.0.7 (released)
with support for: ReadLine, XPM, PNG, Shape, XShm, SM, XRandR, XRender, XCursor, XFT, NLS

  • Linux distribution or BSD name/version

FreeBSD 12.4-STABLE

  • Platform (run: uname -sp)

FreeBSD amd64

Expected Behaviour

Pager appears in indicated (e.g. top left) corner of primary monitor.

Actual Behaviour

Pager appears in indicated (e.g. top left) corner of leftmost monitor.

In situations where the leftmost monitor is smaller than the primary one, this might cause the pager to not be visible at all.

fvwm2 did handle this correctly.

Steps to Reproduce

  • Setup a configuration with more than one monitor, and make any monitor other than leftmost the primary one (using xrandr, arandr etc.)
  • Start the pager

Does Fvwm3 crash?

no

Extra Information

@dl8dtl dl8dtl added the type:bug Something's broken! label Aug 21, 2023
@ThomasAdam
Copy link
Member

Seems to be working just fine for me. Can you share your FvwmPager configuration, and any relevant Style lines you might have in your configuration for FvwmPager?

@dl8dtl
Copy link
Author

dl8dtl commented Aug 21, 2023

Strange. Same behaviour here for the ButtonBox, btw.
The configuration has been migrated from fvwm2 (using fvwm-convert-2.6).

AddToFunc "InitFunction" "I" Module FvwmPager 0 0
AddToFunc "RestartFunction" "I" Module FvwmPager 0 0
…
# (tried both these)
#Style "FvwmPager"  StaysOnTop, !Title, !Handles, Sticky, WindowListSkip
Style "FvwmPager"  !Title, !Handles, Sticky, WindowListSkip
…
########################### Pager #########################################
*FvwmPager: Back #908090
*FvwmPager: Fore #484048
*FvwmPager: Font "xft:MiscFixed-6";-adobe-helvetica-bold-r-*-*-11-*-*-*-*-*-*-*
*FvwmPager: Hilight #cab3ca
*FvwmPager: Geometry +10+10
#*FvwmPager: Label 0 #1
#*FvwmPager: Label 1 #2
#*FvwmPager: Label 2 Mail
#*FvwmPager: Label 3 Matlab
#*FvwmPager: SmallFont 5x8
*FvwmPager: SmallFont "xft:MiscFixed-6";5x8
*FvwmPager: DeskTopScale 30

I think that are all the relevant parts. The config file has been migrated and migrated and migrated … I've been using fvwm for probably 30 years now.

@dl8dtl
Copy link
Author

dl8dtl commented Aug 22, 2023

Maybe it's the "+10+10" that is now misinterpreted? It used to be interpreted to the primary screen before.

@ThomasAdam
Copy link
Member

You can always add @p to the end of the geometry string.

I have other fixes for this though.

@dl8dtl
Copy link
Author

dl8dtl commented Aug 23, 2023

You can always add @p to the end of the geometry string.

OK, that helps. It would make sense then if fvwm-convert-2.6 appended the @p.

@ThomasAdam ThomasAdam added the skip:changelog Issue/PR should skip CHANGELOG label Aug 26, 2023
@ThomasAdam
Copy link
Member

Will reference this here: #878

No code changes required.

@ThomasAdam ThomasAdam closed this as not planned Won't fix, can't repro, duplicate, stale Aug 26, 2023
@dl8dtl
Copy link
Author

dl8dtl commented Aug 26, 2023

Thanks, good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip:changelog Issue/PR should skip CHANGELOG type:bug Something's broken!
Projects
None yet
Development

No branches or pull requests

2 participants