kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #36299
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