unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Iñigo Serna" <inigoserna@gmail.com>
To: emacs-devel@gnu.org
Subject: Advicing select-windows, shr rendering, and per-buffer-theme package
Date: Fri, 18 Mar 2016 12:04:14 +0100	[thread overview]
Message-ID: <87wpp0xd1t.fsf@gmail.com> (raw)

Hi,

I'm the author of “per-buffer-theme”, a simple package available in
MELPA [1], which changes emacs theme when switching to a buffer according to
some configured rules (f.e. use a dark theme when coding, light theme
for email or web navigation). It advices `select-window', checks buffer
name or major-mode, and changes theme.

It worked correctly and fast with emacs 24.5, but now with emacs 25.x it
produces additional “flickering” when switching buffers, especially for
eww/shr buffers, where it makes package useless.

I think problem comes that, when switching to a eww/shr buffer, shr
rendering uses many temporal buffers, so theme changes many times
producing that "flickering" effect. But I'm not sure as my elisp skills
and emacs internals knowledge is quite limited.

I don't know how to solve it… I've tested with before/after variations
to `advice-add' but results are similar. First versions used
`window-configuration-change-hook' but add-advice to select-window was
much better option then. Any ideas?

Thanks in advance,
Iñigo Serna

[1] http://melpa.org/#/per-buffer-theme
    https://bitbucket.org/inigoserna/per-buffer-theme.el



             reply	other threads:[~2016-03-18 11:04 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-18 11:04 Iñigo Serna [this message]
2016-03-18 13:38 ` Advicing select-windows, shr rendering, and per-buffer-theme package Clément Pit--Claudel
2016-03-18 13:40 ` Clément Pit--Claudel
2016-03-18 16:45   ` Iñigo Serna
2016-03-18 17:45     ` Clément Pit--Claudel
2016-03-18 20:17       ` Iñigo Serna
2016-03-18 22:07         ` raman
2016-03-18 13:55 ` raman
2016-03-18 15:21   ` Stefan Monnier
2016-03-18 16:28     ` raman
2016-03-18 16:47       ` Stefan Monnier
2016-03-18 22:13       ` Iñigo Serna
2016-03-18 22:48         ` raman
2016-03-18 17:04     ` Iñigo Serna

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87wpp0xd1t.fsf@gmail.com \
    --to=inigoserna@gmail.com \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).