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

Simplify `golden-ratio-mode' #48

Merged
merged 3 commits into from
May 1, 2015
Merged

Conversation

abo-abo
Copy link
Contributor

@abo-abo abo-abo commented May 1, 2015

No description provided.

abo-abo added 3 commits May 1, 2015 09:33
* golden-ratio.el (golden-ratio-in-progress): New defvar.
(golden-ratio): Use `golden-ratio-in-progress' instead of turning off
the mode.
(select-window): New defadvice; remove all other stuff.
(golden-ratio-mode): Remove all hooks, the only entry point is
`select-window'.

This change will greatly simplify the life of package writers that have
to work around `golden-ratio-mode'.

Re abo-abo/hydra#64.
* golden-ratio.el (golden-ratio--resize-window): Update.

Shrinking selected window may result in enlarging a window that was
supposed to be ignored.
* golden-ratio.el (split-window): New advice.
(golden-ratio): Fixup.
@thierryvolpiatto
Copy link
Collaborator

Thanks, but now golden-ratio is not happening when splitting windows.
Also, golden-ratio-exclude-modes have no more effects.

@abo-abo
Copy link
Contributor Author

abo-abo commented May 1, 2015

Thanks, but now golden-ratio is not happening when splitting windows.

No problem, just needs one more advice.

Also, golden-ratio-exclude-modes have no more effects.

Fixed up, that logic was lost in the merge conflict.

thierryvolpiatto pushed a commit that referenced this pull request May 1, 2015
Simplify `golden-ratio-mode'
@thierryvolpiatto thierryvolpiatto merged commit 7593ba5 into roman:master May 1, 2015
@thierryvolpiatto
Copy link
Collaborator

Ok looks good now, thanks.

@thierryvolpiatto
Copy link
Collaborator

Oleh Krehel [email protected] writes:

Thanks, but now golden-ratio is not happening when splitting windows.

No problem, just needs one more advice.

Also, golden-ratio-exclude-modes have no more effects.

Fixed up, that logic was lost in the merge conflict.

Now scrolling in Gnus is broken...

Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997

@abo-abo
Copy link
Contributor Author

abo-abo commented May 1, 2015

Now scrolling in Gnus is broken...

It works fine for me.

@thierryvolpiatto
Copy link
Collaborator

Oleh Krehel [email protected] writes:

Now scrolling in Gnus is broken...

It works fine for me.

I mean gnus-summary-scroll-up (press RET repetitively from summary
buffer).

Maybe it is working in emacs-25 due to a difference in select-window,
but it is broken in emacs-24.4/5.

Disabling golden-ratio-mode let it working, switching to version 1.0 of
golden ratio fix the problem too.

Please try to fix, otherwise I will look into this ASAP (not the time
now).

Thanks.

Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997

@abo-abo
Copy link
Contributor Author

abo-abo commented May 1, 2015

I mean gnus-summary-scroll-up (press RET repetitively from summary buffer). Maybe it is working in emacs-25 due to a difference in select-window, but it is broken in emacs-24.4/5.

Its scrolls fine in 24.5.2. Although it's weird to have golden-ratio on at all for gnus, since it makes the summary (which I like to keep small) much larger than the article.

@thierryvolpiatto
Copy link
Collaborator

Its scrolls fine in 24.5.2.

No it doesn't. I have opened an issue #49 .

thierryvolpiatto pushed a commit that referenced this pull request May 1, 2015
* golden-ratio.el: Fix issue by reverting PR (#48).
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.

2 participants