From mboxrd@z Thu Jan 1 00:00:00 1970 From: Toby Cubitt Subject: Re: bug#11249: 24.1.50; Overlay with face property causes calendar buffer to scroll Date: Sat, 21 Apr 2012 21:34:06 +0200 Message-ID: <20120421193406.GA1111@c3po.home> References: <8762cuy5rw.fsf@altern.org> Reply-To: Toby Cubitt Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([208.118.235.92]:41877) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SLg4t-00012g-T2 for emacs-orgmode@gnu.org; Sat, 21 Apr 2012 15:34:44 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SLg4p-0005bL-Cb for emacs-orgmode@gnu.org; Sat, 21 Apr 2012 15:34:39 -0400 Received: from sanddollar.geekisp.com ([216.168.135.167]:14312) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SLg4p-0005am-8N for emacs-orgmode@gnu.org; Sat, 21 Apr 2012 15:34:35 -0400 Content-Disposition: inline In-Reply-To: <8762cuy5rw.fsf@altern.org> List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: emacs-orgmode@gnu.org On Fri, Apr 20, 2012 at 01:57:55PM +0200, Bastien wrote: > Toby Cubitt writes: > > > On Mon, Apr 16, 2012 at 05:50:50AM +0300, Eli Zaretskii wrote: > >> > Date: Sun, 15 Apr 2012 23:04:16 +0200 > >> > Cc: 11249@debbugs.gnu.org, emacs-orgmode@gnu.org > >> > From: Toby Cubitt > >> > > >> > The obvious solution is for org-mode to use a face that doesn't enlarge > >> > the characters. > >> > >> Another solution would be to enlarge the calendar window by one line. > > > > I've attached a patch that does exactly this. As expected, it fixes the > > bug for me. > > > > As discussed previously, the alternative solution would be to change the > > default face used to highlight the current date to something that doesn't > > set the :bold attribute (which would probably mean introducing a separate > > face for this, instead of reusing org-warning). But expanding the > > calendar window is a more general solution, as it fixes the problem for > > (almost) any face. > > I introduced a new face `org-date-selected' which is now the one in use > in the calendar and which is not bold. Thanks for the fix. I notice that the new face's dosctring still says "Face for deadlines and TODO keywords", which needs changing. Was there some reason to reject Eli's suggested fix of enlarging the calendar window by 1 line? As it stands, if anyone customizes the new org-date-selected face to be bold (the highlighted date *is* a little less visible now), it will cause the same problem as before. And the fact that it's caused by making the font bold is not at all obvious...at least, it wasn't obvious to me! Eli's fix works more generally. Anyway, since I now know how to patch things if I do decide I want a bold font, it's not a big issue for me. In the patch I posted, I also took the opportunity to set `cursor-type' to nil when opening the calendar in `org-read-date'. The cursor obscures one digit of the selected date, making that bit harder to read (as well as looking ugly). Did you intend to reject this change too? Toby -- Dr T. S. Cubitt Mathematics and Quantum Information group Department of Mathematics Complutense University Madrid, Spain email: tsc25@cantab.net web: www.dr-qubit.org