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

Add a retile() function. #12

Merged
merged 1 commit into from
Jan 31, 2015
Merged

Add a retile() function. #12

merged 1 commit into from
Jan 31, 2015

Conversation

mavant
Copy link
Contributor

@mavant mavant commented Jan 31, 2015

No description provided.

Since mjolnir doesn't have events yet, this is slightly more convenient
than cycling twice to get everything back to my desired layout.
@csaunders
Copy link
Collaborator

Code looks good. I'd just love to see more of a justification and an example of where you'd use it.

@mavant
Copy link
Contributor Author

mavant commented Jan 31, 2015

Oh, sorry - there was an explanation in the commit message that didn't end
up on the PR. I'm using this as a stopgap measure until mjolnir gets the
ability to listen to window manager events. Have a keybinding that just
does this is a bit faster than hitting mash+c three times whenever I open a
new window.

On Fri, Jan 30, 2015, 20:11 Christopher Saunders notifications@github.com
wrote:

Code looks good. I'd just love to see more of a justification and an
example of where you'd use it.


Reply to this email directly or view it on GitHub
#12 (comment)
.

@csaunders
Copy link
Collaborator

coolbeans

csaunders added a commit that referenced this pull request Jan 31, 2015
Add a retile() function.
@csaunders csaunders merged commit c278fe3 into nathankot:master Jan 31, 2015
@nathankot
Copy link
Owner

@mavant nice 👍 side note, I've been using tiling.promote() to achieve the same thing, but there are definitely cases where this will prove better

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.

None yet

3 participants