Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/image/font/gofont/gomedium: wrong shape for "l" letter #23497

Closed
MichaelMonashev opened this issue Jan 21, 2018 · 11 comments
Closed

x/image/font/gofont/gomedium: wrong shape for "l" letter #23497

MichaelMonashev opened this issue Jan 21, 2018 · 11 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@MichaelMonashev
Copy link

What version of Go are you using (go version)?

go version go1.8.3 windows/386

Does this issue reproduce with the latest release?

I don`t know. I have not new os to install latest release.

What operating system and processor architecture are you using (go env)?

set GOARCH=386
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=386
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\dev
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_386
set GCCGO=gccgo
set GO386=
set CC=gcc
set GOGCCFLAGS=-m32 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\1E86~1\AppData\Local\Temp\go-build017858111=/tmp/go
rd-gcc-switches
set CXX=g++
set CGO_ENABLED=1
set PKG_CONFIG=pkg-config
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2

What did you do?

Use golang.org/x/image/font/gofont/gomedium font for drawing.

Source code, that generate picture with wrong "l" letter shape: https://pastebin.com/eKaQJr0c
Picture: https://imagebin.ca/v/3ozP9RuDTh96

What did you expect to see?

Perfect picture

What did you see instead?

I see "l" letter with wrong shape.

@gopherbot gopherbot added this to the Unreleased milestone Jan 21, 2018
@MichaelMonashev MichaelMonashev changed the title x/image: x/image: wrong shape for "l" letter in gomedium font Jan 21, 2018
@ALTree ALTree added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Jan 21, 2018
@odeke-em
Copy link
Member

/cc @nigeltao @robpike

@odeke-em odeke-em changed the title x/image: wrong shape for "l" letter in gomedium font x/image/font/gofont/gomedium: wrong shape for "l" letter Jan 22, 2018
@nigeltao
Copy link
Contributor

Thanks for the bug report. I've passed it on to Bigelow & Holmes.

@nigeltao nigeltao self-assigned this Jan 24, 2018
@ghost
Copy link

ghost commented Nov 1, 2019

Is there a rough ETA on a fix?

Also experiencing this with Go Medium letter "l" on Linux.

@nigeltao
Copy link
Contributor

nigeltao commented Nov 4, 2019

I'll poke Bigelow & Holmes again about it. If they're still busy, we (the Go team) may be able to issue our own fix to their fonts. I'll get back to you...

@barmadrid
Copy link

@nigeltao any response from Bigelow & Holmes on this?

@nigeltao
Copy link
Contributor

They were busy, but have coincidentally become less so recently. We'll hopefully release a fix soon.

@barmadrid
Copy link

Hi @nigeltao just wondering if you are working on a fix anytime soon?

@nigeltao
Copy link
Contributor

It would be B&H working on a fix. They were optimistic in February about finding some time but unfortunately COVID-19 ruined everyone's plans. I don't have a timeline. Sorry.

@barmadrid
Copy link

Hi @nigeltao Any progress on this yet?

@nigeltao
Copy link
Contributor

I'll ask B&H again.

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/412654 mentions this issue: font/gofont: upgrade to version 2.010

@dmitshur dmitshur added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Jun 17, 2022
@golang golang locked and limited conversation to collaborators Jun 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

7 participants