all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ben Key <bkey76@gmail.com>
To: Chong Yidong <cyd@stupidchicken.com>, Emacs-devel@gnu.org
Subject: Re: Bar Cursor Still To Narrow on Mac OS X
Date: Sat, 5 Mar 2011 20:45:15 -0600	[thread overview]
Message-ID: <AANLkTim8Xi2y0W5TGPuLNVsgbgp0z21V3_+_K_MJjbz3@mail.gmail.com> (raw)
In-Reply-To: <87pqq5f7ga.fsf@stupidchicken.com>

[-- Attachment #1: Type: text/plain, Size: 1397 bytes --]

Thanks.

I tested the latest change with cursor-type set to (bar . 4) and everything
works as expected.  I can now actually see the bar cursor without straining.

Note that with the most recent changes, the cursor is not drawn correctly
when the user sets the cursor-type variable to (hbar . HEIGHT).  According
to the documentation for the cursor-type variable, when this form is used,
the number after the . is supposed to specify the "height" of the cursor,
not the "width."

The function get_specified_cursor_type processes this form by setting the
width output parameter to the user specified width.  The value assigned to
the width output parameter in get_specified_cursor_type is eventually used
to set the value of the cursor_width parameter when ns_draw_window_cursor is
called.

The expectation is that a horizontal bar cursor will be used whose height is
set to the user specified height and width is set to be equal to the width
of the current glyph at point.  This is what happens in Emacs 23.

The problem with the latest code in ns_draw_window_cursor is that the user
specified value for the cursor "height" is currently also being used for the
cursor "width."  As a result, when cursor-type is set to (hbar . 4) the
result is a 4 pixel by 4 pixel block, not a block that is 4 pixels tall and
as wide as the current glyph at point.

I am working on a patch that fixes this problem.

[-- Attachment #2: Type: text/html, Size: 1630 bytes --]

  reply	other threads:[~2011-03-06  2:45 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-05 23:41 Bar Cursor Still To Narrow on Mac OS X Ben Key
2011-03-05 23:55 ` Chong Yidong
2011-03-06  2:45   ` Ben Key [this message]
2011-03-06 20:24     ` Chong Yidong

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

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

  git send-email \
    --in-reply-to=AANLkTim8Xi2y0W5TGPuLNVsgbgp0z21V3_+_K_MJjbz3@mail.gmail.com \
    --to=bkey76@gmail.com \
    --cc=Emacs-devel@gnu.org \
    --cc=cyd@stupidchicken.com \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.