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

fixed split #30

Merged
merged 1 commit into from
Aug 15, 2015
Merged

fixed split #30

merged 1 commit into from
Aug 15, 2015

Conversation

taiyu-len
Copy link
Collaborator

when focus is on workspace, and there is more then 1 child, splitting will now split properly.
it now acts as if the container is created around workspace, without actually doing so.

┏━┯━┓
┃ │ ┃
┗━┷━┛
with vertical split (+ new view) becomes. before it just changes workspace layout.
┏━┯━┓
┠─┴─┨
┗━━━┛

although the effects of this cant be seen yet as focusing still needs to be fixed. (i did test it with a buggy focus fix, but im not pushing that)

what we are currently using to set the focus focus_view doesnt support focus in the middle of the layout tree, so when workspace has children, focus cant be on workspace.

oh and i also moved the if(parent->focused==child)parent->focused=NULL); into the free_swayc function.

@ddevault
Copy link
Contributor

LGTM

ddevault added a commit that referenced this pull request Aug 15, 2015
@ddevault ddevault merged commit 648675c into swaywm:master Aug 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants