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

PositionPlacement Center: fix to use current screen #211

Closed
ThomasAdam opened this issue Sep 8, 2020 · 1 comment · Fixed by #212
Closed

PositionPlacement Center: fix to use current screen #211

ThomasAdam opened this issue Sep 8, 2020 · 1 comment · Fixed by #212
Assignees
@ThomasAdam
Labels
type:bug Something's broken!
Milestone
1.0.1

Comments

@ThomasAdam
Copy link
Member

ThomasAdam commented Sep 8, 2020 •
edited
Loading

When using Style foo PositionPlacement Center, the screen parameter isn't being used, meaning this has no effect on the window.
@issue-label-bot issue-label-bot bot added the type:bug Something's broken! label Sep 8, 2020
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label bug to this issue, with a confidence of 0.94. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@ThomasAdam ThomasAdam self-assigned this Sep 8, 2020
@ThomasAdam ThomasAdam added this to the 1.0.1 milestone Sep 8, 2020
ThomasAdam added a commit that referenced this issue Sep 8, 2020
@ThomasAdam
PositionPlacement: include screen for Center
d71ae23 
When using "PositionPlacement Center", take into account the screen the
window is on, so it's centered appropriately.

Fixes #211
@ThomasAdam ThomasAdam mentioned this issue Sep 8, 2020
Merged
ThomasAdam added a commit that referenced this issue Sep 8, 2020
@ThomasAdam
PositionPlacement: include screen for Center
71c5785 
When using "PositionPlacement Center", take into account the screen the
window is on, so it's centered appropriately.

Fixes #211
mikeandmore pushed a commit to mikeandmore/fvwm3 that referenced this issue Nov 28, 2020
@ThomasAdam @mikeandmore
PositionPlacement: include screen for Center
9687593 
When using "PositionPlacement Center", take into account the screen the
window is on, so it's centered appropriately.

Fixes fvwmorg#211
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.1
Development

Successfully merging a pull request may close this issue.

PositionPlacement: include screen for Center fvwmorg/fvwm3
1 participant
@ThomasAdam