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

Incorporate transpose-windows #18

Closed
dgtized opened this issue Feb 12, 2016 · 2 comments
Closed

Incorporate transpose-windows #18

dgtized opened this issue Feb 12, 2016 · 2 comments

Comments

@dgtized
Copy link
Contributor

dgtized commented Feb 12, 2016

https://www.emacswiki.org/emacs/TransposeWindows

crux has swap-windows, which is for two windows only, and appears to require some special tricks around window-start and the like.

I've had it in my .config for years, and it's quite useful (particularly on larger displays), but otherwise only seems to be directly referenced in the emacs wiki. I'm happy to make a PR to pull it in, but wanted to check if anyone had ideas on how to handle attribution correctly. This package seems an appropriate place to bring it in, but it's definitely one of those snippets that's tricky to determine who wrote it originally, so not sure if that is a concern.

@bbatsov
Copy link
Owner

bbatsov commented Feb 12, 2016

PR welcome.

This package seems an appropriate place to bring it in, but it's definitely one of those snippets that's tricky to determine who wrote it originally, so not sure if that is a concern.

I think that a simple comment will do. I definitely don't think this is a major concern. GPL is known as copyleft for a reason. :-)

@dgtized dgtized changed the title Incorporate tranpose-windows Incorporate transpose-windows Feb 14, 2016
@dgtized
Copy link
Contributor Author

dgtized commented Feb 14, 2016

Addressed by #21 and #22.

@bbatsov bbatsov closed this as completed Feb 14, 2016
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

No branches or pull requests

2 participants