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

feat: limit windows per app #3481

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

decodism
Copy link
Contributor

It could be useful to limit the number of windows in a shortcut with all the apps. A shortcut with the active app can still be used if necessary.

@lwouis
Copy link
Owner

lwouis commented Jul 11, 2024

Hi,

Thank you for sharing this PR.

I think this PR overlaps a bit with #337. I think it would appeal to user who want some kind of group-per-app, hierarchy, or drilldown capability. Of course it's more crude since it's truncating data. So it would help only a subset of the people looking for #337.

As it stands, I think it adds to the issues with #351. I think it's interesting, but not comprehensive enough to warrant adding more settings, as it stands. @Allsochen is currently working on #351. Do you have an opinion on this perhaps @Allsochen?

Thank you

@decodism
Copy link
Contributor Author

This could perhaps be a stopgap solution before what's planned.

@lwouis
Copy link
Owner

lwouis commented Jul 18, 2024

This could perhaps be a stopgap solution before what's planned.

@Allsochen is working on #3468 currently. I think the long-term #337 or a stop-gap workaround should be thought in the context of this work. In another words, if we would merge MR this today, we would need to add migrating it to the new preferences UI in #3468

@lwouis lwouis force-pushed the master branch 2 times, most recently from 7d7d9cf to 8abb9b4 Compare October 10, 2024 09:28
@lwouis lwouis force-pushed the master branch 2 times, most recently from d0d2314 to 0b198b4 Compare December 1, 2024 21:18
@lwouis lwouis force-pushed the master branch 2 times, most recently from baa411d to 987b6b2 Compare December 8, 2024 11:31
@lwouis lwouis force-pushed the master branch 2 times, most recently from 18ef16f to e07da9c Compare December 26, 2024 23:36
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.

2 participants