wayfire icon indicating copy to clipboard operation
wayfire copied to clipboard

DRAFT scale: include all views visible on workspace

Open mntmn opened this issue 2 years ago • 2 comments

Intuitively, "scale" should include all views that are visible in (overlap with) the current workspace, not only those that are mostly inside of it.

mntmn avatar Apr 14 '23 20:04 mntmn

@soreau Thoughts?

By the way, if we change the behavior as you propose, the whole function can be replaced with a single call:

output->wset()->get_views(wf::WSET_EXCLUDE_MINIMIZED | wf::WSET_MAPPED_ONLY | wf::WSET_CURRENT_WORKSPACE);

EDIT: this is with some changes which recently landed, this syntax wasn't available when you made the PR.

ammen99 avatar Jun 09 '23 11:06 ammen99

I think the behavior should be configurable to allow users to choose between old and new behavior.

Probably it should hide away views which region overlaps only a little with the region of the current workspace.

NamorNiradnug avatar Jul 07 '23 12:07 NamorNiradnug