Re: Fringes again

[Simon Josefsson <jas@extundo.com>]

Miles Bader <miles@gnu.org> writes:

> Simon Josefsson <jas@extundo.com> writes:
>> Does anyone have any opinions on the attached code?
>
> Looks good to me.
>
> A few little nits:
>
>   * I think you should use `left-only' and `right-only' instead of
>     `leftonly' and `rightonly' -- they're easier to read and more
>     consistent with general elisp style.
>
>   * It would be useful if the doc-strings for `set-fringe-style' and
>     `set-fringe-mode' referenced each other explicitly, since the names
>     are so similar that they may cause some confusion.

Fixed.  I committed the code.

>> I have one remaining question: How do you find out the default fringe
>> width?  Right now it is hard coded to 10.
>
> I thought (without having looked at the code) it was 8 pixels rounded
> up to a multiple of the width of a character in the default font.

Right, but how do you programmatically calculate that?