[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#37578: closed (Re: bug#37578: 26.3; image-mode does not display the
From: |
Eli Zaretskii |
Subject: |
bug#37578: closed (Re: bug#37578: 26.3; image-mode does not display the bottom edge of a tall image) |
Date: |
Wed, 23 Oct 2019 20:58:15 +0300 |
> From: Tassilo Horn <tsdh@gnu.org>
> Cc: ynyaaa@gmail.com, 37578@debbugs.gnu.org, Eli Zaretskii <eliz@gnu.org>
> Date: Wed, 23 Oct 2019 18:07:53 +0200
>
> > image-mode.el was scrolling images vertically in units of canonical
> > character height, so it couldn't handle the last portion of an image
> > smaller than the height of one character.
> >
> > This is now fixed on the master branch.
>
> Your change 9c66b09950cf2db50eb6d818656a268ef750bfe6 broke doc-view.el
> and the pdf-tools package whose author already filed bug#37874 for the
> issue.
>
> Could you have a look? If the image functions stay returning pixel
> values that's ok but Andreas and I will need to adapt pdf-tools and
> doc-view.el accordingly.
Sorry, I didn't think some other package will use that function, it
looked like an internal subroutine. I should have checked.
To answer your question: I see no reason to go back to character units
in this function. We could either adapt the other packages to the
change, or add a new function that will work like the previous
image-next-line. What would you prefer? If the only problem is with
comparing against the value returned by window-vscroll, then perhaps
the former is easier and simpler?
Thanks.