Add option to limit to outputs

[Syphdias]

To enable different limits on different outputs, you can start two processes and limit them to the outputs you want.

Like suggested in #41.
I did not implement anything for nwg-panel to not break anything.

[nwg-piotr]

Thanks! I'll test as soon as I'm at my own desk.

[nwg-piotr]

At least on sway

output = e.ipc_data.get("container", {}).get("output", "")

returns nothing. No "output" in ipc_data.

[Syphdias]

I couldn't take a look into in yet. Hm, maybe there is a more reliable way to get the focused output.

[nwg-piotr]

I used some recursion in nwg-autotiling.

[Syphdias]

It's so easy with the workspace... i3.get_tree().find_focused().workspace()

With this, I think I found another way. Iterate over outputs and check if the current workspace is active on it.

def output_name(i3):
    focused_container = i3.get_tree().find_focused()
    for output in i3.get_outputs():
        if focused_container.workspace().name == output.current_workspace:
            return output.name
    return None

I measured both approaches and scanning the parents is a lot faster than collectin information about the outputs.

[Syphdias]

I changed how the output is being detected. Could you give it another try?

[nwg-piotr]

It seems to work well. Some minor suggestions:

parser.add_argument("-o",
                        "--outputs",
                        help="restricts autotiling to certain output; "
                        "example: autotiling --output  DP-1 HDMI-0",
                        nargs="*",
                        type=str,
                        default=[], )

This only affects a single output. Shouldn't it be --output?

help='limit how often autotiling will split a container;

"How often" sounds misleading to me. Maybe just "how deep"?

[Syphdias]

This only affects a single output. Shouldn't it be --output?

No, outputs is correct. It works very similar to --workspaces. It can takes multiple values.

"How often" sounds misleading to me. Maybe just "how deep"?

I am a bit confused the patch does not contain this. I think you are referring to the --limit option.

[nwg-piotr]

I must've made a mistake, sorry.

[Syphdias]

Feel free to rephrase the help though! I was struggling with finding a good way to describe it.

[nwg-piotr]

So was I. I wouldn't be easy to a native English speaker, I bet.