search

TimUntersberger / nog

A tiling window manager for Windows
MIT License
697 stars 20 forks source link

fix(#279) update components to display per workspace and sort ids in … #280

Closed ramirezmike closed 3 years ago

ramirezmike commented 3 years ago

…workspace component

ramirezmike commented 3 years ago

@TimUntersberger

I made a few changes here to make these components behave more like I think that they used to. I remember the fullscreen and split used to be display specific at least.

Let me know what you think about the changes and also if you want this to wait until after #277 that's totally cool, no worries.

TimUntersberger commented 3 years ago

Nice! Could you move the implementation of both new functions into AppState so we can more easily reuse it?

Everything else is LGTM

ramirezmike commented 3 years ago

Moved get_focused_ws_of_display and get_focused_win_of_display into AppState and will merge once checks have passed