← Back to team overview

kicad-developers team mailing list archive

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

 

Looks very good to me. I could nitpick aesthetics (I think it might be a
_little_ bit too long now) but that's just useless bikeshedding - it's
very clear and visible and balanced.

On Sat, Jun 23, 2018 at 12:00:04PM -0400, Wayne Stambaugh wrote:
> 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
> 
> _______________________________________________
> 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


References