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

Grouped window list: Add option to only list windows from the current monitor #12225

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

jarierca
Copy link
Contributor

@jarierca jarierca commented Jun 8, 2024

Hi :)

I added an old feature to show only active applications on the current monitor which was deleted in commit (ef4610c)
Now allows to toggle between showing only current monitor applications or showing all apps from all monitors. This PR is related to this topic #8802

@JosephMcc
Copy link
Contributor

Imho, this functionality should never have been removed. It's one of the most useful features when using multiple monitors.

@clefebvre
Copy link
Member

How do you use it though?

As it is, it already does only show you your monitor apps if you have an applet on the other monitor.

@jarierca
Copy link
Contributor Author

Hello, I'm sorry for explaining so briefly.
As it is now, if you have 1 panel and for example 3 monitors, all applications are displayed in the single panel.

only-1-panel

However, if you want to have 3 panels (1 per monitor), applications from all monitors are not displayed.

only-list-apps-from-current-monitor

That's why I added the button "Only list windows in the current monitor" (when you have more than 1 panel) (defaulting to true, which would have the same effect as it currently is before this change).

grouped-window-list-config

If I set this button to false, for example on the central monitor it would look like this (if I tried to do the same as it is now, I wouldn't see all the apps open on the central monitor).

list-apps-from-all-monitor-mid-monitor

And if I mark it as false in the configuration for all panels, it would look like this:
list-apps-from-all-monitors

If you need more help or information, let me know. ;)

@mtwebster
Copy link
Member

Hi, seems to work fine, I'm just not sure about the wording of the setting - when you compare it to the current behavior it can be a bit ambiguous, depending on your current layout. I think maybe it could be better as a combo box:

Multiple instances:

  • Track windows individually per monitor
  • Always track windows from all monitors

The first being the default - it doesn't really describe the edge case of having two instances but three monitors, but I don't think that's common anyhow.

I'm not set on this, just throwing an idea out

… two most common options to improve clarity in the use of window display mode.
@jarierca
Copy link
Contributor Author

Hi @mtwebster

Thank you for your feedback!. I have updated the settings to include a combo box with the two most common options: "Track windows individually per monitor" and "Track windows across all monitors." This change aims to improve clarity in the use of window display mode.

Thanks again for your input!

@mtwebster mtwebster merged commit 3e1fca6 into linuxmint:master Nov 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants