<p><br>
On Jun 23, 2012 1:34 AM, "David Bremner" <<a href="mailto:david@tethera.net">david@tethera.net</a>> wrote:<br>
><br>
> Jani Nikula <<a href="mailto:jani@nikula.org">jani@nikula.org</a>> writes:<br>
><br>
> ><br>
> > A small wrinkle is that in order to make the variable survive<br>
> > notmuch-show-refresh-view (which is required for expanding/collapsing<br>
> > the parts, but ends up calling kill-all-local-variables through<br>
> > notmuch-show-mode) it is necessary to give it the permanent-local<br>
> > property.<br>
><br>
> The code looks simple enough; should we apply this patch while we wait<br>
> for something fancier?</p>
<p>Applying this doesn't make the fancy stuff harder to do at all. And this doesn't have a key binding, on purpose, so we're not committed to that either. But then I'm biased...</p>
<p>> I don't really know how to evaluate the<br>
> permanent-local bit.</p>
<p>All the other show mode toggles use that too. It's probably less surprising like this than the permanently local only as needed approach in v1.</p>
<p>><br>
> d<br>
</p>