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

FvwmPager: remove list_focus handling #964

Merged
merged 1 commit into from
Mar 18, 2024
Merged

Conversation

ThomasAdam
Copy link
Member

This was only ever used internally by FvwmPager to keep track of which
monitor is currently focused. This is no longer required now that this
information is used directly from libfvwm3.

This was only ever used internally by FvwmPager to keep track of which
monitor is currently focused.  This is no longer required now that this
information is used directly from libfvwm3.
@ThomasAdam ThomasAdam added this to the 1.1.0 milestone Mar 18, 2024
@ThomasAdam ThomasAdam self-assigned this Mar 18, 2024
@ThomasAdam ThomasAdam added type:bug Something's broken! relates:module Issue is in module code labels Mar 18, 2024
@ThomasAdam ThomasAdam merged commit 0ebd9dc into main Mar 18, 2024
5 checks passed
@ThomasAdam ThomasAdam deleted the ta/remove-fvwmpager-debug branch March 18, 2024 22:58
@somiaj
Copy link
Collaborator

somiaj commented Mar 18, 2024

Why not just remove the whole function (vs it do nothing)?

@ThomasAdam
Copy link
Member Author

I'm about to add different functionality to that, but it might not be in the 1.1.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relates:module Issue is in module code type:bug Something's broken!
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants