all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Evgeny Zajcev <lg.zevlg@gmail.com>
To: Stefan Kangas <stefankangas@gmail.com>
Cc: Eli Zaretskii <eliz@gnu.org>,
	Stefan Monnier <monnier@iro.umontreal.ca>,
	emacs-devel <emacs-devel@gnu.org>
Subject: Re: [PATCH] Fix display box cursor under image with margins
Date: Sun, 10 May 2020 17:08:50 +0300	[thread overview]
Message-ID: <CAO=W_ZpQ-_E35RwENw4Xm6k8=Bbo4wk_y5BGk58S5MCQe=55Ug@mail.gmail.com> (raw)
In-Reply-To: <CADwFkmk5TswbM7mMyD2uO3wstOXwc_2VKaZ3DoG5f5eem+1G9g@mail.gmail.com>

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

вс, 10 мая 2020 г. в 15:52, Stefan Kangas <stefankangas@gmail.com>:

> Evgeny Zajcev <lg.zevlg@gmail.com> writes:
>
> > сб, 25 апр. 2020 г. в 15:09, Stefan Monnier <monnier@iro.umontreal.ca>:
> >
> >> > Here is first screenshot how hollow cursor looks on image with margin
> -
> >> > http://lgarc.narod.ru/pics/emacs-hollow-cursor.png
> >> >
> >> > And here is how it looks with patch applied -
> >> > http://lgarc.narod.ru/pics/emacs-box-cursor.png
> >>
> >> Would it be possible to keep the two thin horizontal lines when the
> >> window is active?
> >>
> > I figured out, how to do this in "easy" way.  Here is patches update
> >
> > And screenshots: http://lgarc.narod.ru/pics/emacs-box-margin1.png
> > http://lgarc.narod.ru/pics/emacs-box-margin2.png
>
> I think your second set of screenshots looks better.
>

On second screeshot, image has vertical margin as (10 . 4) specifies, that
is why horizontal lines are not thin

On first screenshot, there is only hmargin as (10 . 0) specifies


> But the horizontal lines are not thin, right?
>
> What happens when you switch the active window?  Will the cursor change
> the size of that line, causing things to move vertically?  And what's
> the result when you have many images in a buffer, as in image-dired?
>

No moves, margins are part of the image size displayed in Emacs window.
Cursor just uses that margins to fulfill with cursor color

In both screenshots below window shows how it looks in inactive window.  It
looks unchanged from how they look in current Emacs without patch

I filmed short video how it looks in dynamic -
http://lgarc.narod.ru/pics/IMG_9721.MOV

-- 
lg

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

  reply	other threads:[~2020-05-10 14:08 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-18 17:28 [PATCH] Fix display box cursor under image with margins Evgeny Zajcev
2020-04-18 17:44 ` Eli Zaretskii
2020-04-18 18:05   ` Evgeny Zajcev
2020-04-25 10:35     ` Eli Zaretskii
2020-04-25 10:48       ` Evgeny Zajcev
2020-04-25 10:55       ` Evgeny Zajcev
2020-04-25 11:22       ` Basil L. Contovounesios
2020-04-25 13:19       ` Clément Pit-Claudel
2020-04-25 12:09     ` Stefan Monnier
2020-04-25 12:44       ` Evgeny Zajcev
2020-04-25 13:57       ` Evgeny Zajcev
2020-05-10 12:14         ` Evgeny Zajcev
2020-05-10 14:09           ` Eli Zaretskii
2020-06-13 15:55             ` Basil L. Contovounesios
2020-07-24 11:45               ` Stefan Kangas
2020-05-10 12:52         ` Stefan Kangas
2020-05-10 14:08           ` Evgeny Zajcev [this message]
2020-05-10 15:38             ` Michael Welsh Duggan
2020-05-10 16:48               ` Stefan Monnier
2020-05-11 15:51             ` Stefan Kangas
2020-05-11 16:34               ` andres.ramirez

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='CAO=W_ZpQ-_E35RwENw4Xm6k8=Bbo4wk_y5BGk58S5MCQe=55Ug@mail.gmail.com' \
    --to=lg.zevlg@gmail.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=stefankangas@gmail.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.