← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Fix overbar drawing for non-italic text

 

This looks correct to me. Is there something I'm missing here? I don't pretend to be an expert on character rendering.

On 06/23/2018 11:32 AM, Andrzej Wolski wrote:
Wayne,

this is after patch:
https://imgur.com/a/54ESCMD

Andrzej

W dniu 2018-06-23 o 16:54, Wayne Stambaugh pisze:
Andrzej,

Do you have an image of the same characters after you patch?  The other
question I have is what happens when consecutive characters have and
overbar?  AFAIK, the overbar should be continuous (no breaks between
characters).  Did you test this as well.  If this needs fixed, I would
like to get it merged for rc3.  Anyone else have any thoughts on this.

Cheers,

Wayne

On 06/22/2018 10:35 AM, Andrzej Wolski wrote:
No, the patch didn't go the wrong way.
If you try to shorten the right side of the bar, for some characters it
will look really bad, and in case of "I" it will turn into dot.
This is before patch:
https://imgur.com/a/TjibR0Q

Andrzej


W dniu 2018-06-22 o 15:40, Kevin Cozens pisze:
On 2018-06-22 07:01 AM, Andrzej Wolski wrote:
This is an illustration of what this patch do:
https://imgur.com/a/GxIeLkA
Hm... did the patch go the wrong way with the change to the position
and length of the bar over non-italicized text? Before the bar started
at the left side of the letter and extends past the right side of the
letter. After it starts before the letter and extends past the right
side. Contrast that to italicized text where the bar starts and ends
based on the left-most and right-most part of the drawn letter.

I think a patch should shorten the right-hand extent of the bar over
non-italicized text so it is only over the letter.


_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp
_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp



_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Follow ups

References