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

mu4e--view-render-buffer: Disable Gnus bookmark-make-record-function #2672

Merged

Conversation

alphapapa
Copy link
Contributor

The function 'gnus-summary-bookmark-make-record' does not work properly with the
faux "appeasement" summary buffer, causing undesired changes in the window
configuration when the message rendering buffer's bookmark function is
called (which some packages, like Activities, Burly, and Dogears do routinely).

See alphapapa/activities.el#55.

Reported-by: Daniel Goldin https://github.com/danielgoldin

The function 'gnus-summary-bookmark-make-record' does not work properly with the
faux "appeasement" summary buffer, causing undesired changes in the window
configuration when the message rendering buffer's bookmark function is
called (which some packages, like Activities, Burly, and Dogears do routinely).

See <alphapapa/activities.el#55>.

Reported-by: Daniel Goldin <https://github.com/danielgoldin>
@djcb djcb merged commit e4f2dd7 into djcb:release/1.10 Mar 27, 2024
@djcb
Copy link
Owner

djcb commented Mar 27, 2024

Merged, thanks. Does this apply to 1.12.x as well?

@alphapapa
Copy link
Contributor Author

Merged, thanks. Does this apply to 1.12.x as well?

Thanks.

Yes, it appears to apply there as well. (I didn't see a release branch for the 1.12 series, so I thought 1.10 was the latest stable series. How does this part of mu's release process work? I didn't see it in the docs but I may have overlooked it.)

@djcb
Copy link
Owner

djcb commented Mar 29, 2024

Ah, until there are some big / incompatible changes, we do 1.12 releases straight from master. Probably should document that somewhere, indeed!

@alphapapa
Copy link
Contributor Author

Ok, do I need to send a PR for 1.12 or can you cherry-pick it?

@djcb
Copy link
Owner

djcb commented Mar 29, 2024

Oh, no worries, I'll cherry-pick in a few days (after travels). Thanks!

@alphapapa
Copy link
Contributor Author

@djcb Hi again,

I see that on master you merged 7688a2b which no longer "appeases" Gnus. Do you know if that also solves this problem, i.e. preventing Gnus's bookmark-make-record-function from changing the window configuration?

@djcb
Copy link
Owner

djcb commented Apr 9, 2024

@alphapapa: hopefully so, but I am not 100% sure. Some light testing didn't seem to trigger the problem with "activities".

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

2 participants