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

Home page shows the project title as "dorkq" on some Android devices #81

Closed
drfloob opened this issue Oct 27, 2013 · 34 comments
Closed

Home page shows the project title as "dorkq" on some Android devices #81

drfloob opened this issue Oct 27, 2013 · 34 comments

Comments

@drfloob
Copy link

@drfloob drfloob commented Oct 27, 2013

See here:
http://www.browserstack.com/screenshots/2308d26f99f0b5915b53829c4ae5f9567b2d84be

I haven't tried to figure out why. @practicum noticed that the letters are all 3 off and lowercase. G-3=d, R-3=o ...

@vladikoff
Copy link
Member

@vladikoff vladikoff commented Oct 27, 2013

@drfloob Thanks for the report and a screenshot, we will try to figure out why this happening.

Test example:

image

@shama
Copy link
Member

@shama shama commented Oct 27, 2013

We could just rename grunt to dorkq to fix this problem, no?

@vladikoff
Copy link
Member

@vladikoff vladikoff commented Oct 27, 2013

@shama, no we need to rename to juxqw

@shama
Copy link
Member

@shama shama commented Oct 27, 2013

@vladikoff aww, never mind then. I dont want to type that.

@vladikoff
Copy link
Member

@vladikoff vladikoff commented Oct 27, 2013

The issue is

-webkit-text-stroke: 4px;

fixing...

@vladikoff
Copy link
Member

@vladikoff vladikoff commented Oct 27, 2013

Fixed. Motorola Droid 4 users are safe from the evil dorkq!
image

@vladikoff vladikoff closed this Oct 27, 2013
@zackbloom
Copy link

@zackbloom zackbloom commented Oct 28, 2013

After looking at 858d22a, it looks like the problem was actually the font (makes more sense than it being text-stroke)?

@ghepting
Copy link

@ghepting ghepting commented Oct 28, 2013

That is awesome, lol...

@vladikoff
Copy link
Member

@vladikoff vladikoff commented Oct 28, 2013

@zackbloom I think I had to choose to either keep text-stroke or keep the h1

@moeffju
Copy link

@moeffju moeffju commented Oct 28, 2013

@vladikoff You wouldn’t happen to have a reduced testcase or a minimal changeset? :)

@zackbloom
Copy link

@zackbloom zackbloom commented Oct 29, 2013

Yea, I'd love an explanation on what the actual bug was.

@vladikoff
Copy link
Member

@vladikoff vladikoff commented Oct 29, 2013

image

It's some combination of text-stroke being inside of an h1. So you have to choose either have an h1 or text-stroke. I tried making an isolated test case but can't reproduce, I don't have any of the devices that had this problem.

@canweriotnow
Copy link

@canweriotnow canweriotnow commented Oct 29, 2013

added-pony

(Really, just wanted to comment on epic issue)

@RobLoach
Copy link

@RobLoach RobLoach commented Oct 29, 2013

There is no Dana, only dorkq.

@kahlil
Copy link

@kahlil kahlil commented Oct 29, 2013

I am sad that dorkq is taken on Twitter :(

Sent from my iPhone

On 29.10.2013, at 12:52, Rob Loach notifications@github.com wrote:

There is no Dana, only dorkq.


Reply to this email directly or view it on GitHub.

@RobLoach
Copy link

@RobLoach RobLoach commented Oct 29, 2013

dorkq

@Boldewyn
Copy link

@Boldewyn Boldewyn commented Oct 29, 2013

Is that a font issue? The string "dorkq" is not random:

d - e - f - g
o - p - q - r
r - s - t - u
k - l - m - n
q - r - s - t
@kahlil
Copy link

@kahlil kahlil commented Oct 29, 2013

@Boldewyn nice find!

@yahuarkuntur
Copy link

@yahuarkuntur yahuarkuntur commented Oct 29, 2013

And if we sum it up with random numbers... the result is 42!

@cheeaun
Copy link

@cheeaun cheeaun commented Oct 29, 2013

For anyone interested, I've made a reduced test case here http://jsbin.com/aMuzAn/7

Browserstack screenshots: http://www.browserstack.com/screenshots/42f41255e13b5dce1cbdcae58097900eedf9c2ee

@andrewsf
Copy link

@andrewsf andrewsf commented Oct 29, 2013

@Boldewyn @kahlil That correlation was mentioned in the original report. More likely I think they are off by 29: GHIJKLMNOPQRSTUVWXYZ[]^_`abcd.

Can bug reports on GitHub be "archived" or "frozen" for posterity purposes? I think this one seems mature and—somewhat hypocritically—I don't believe it needs more outside input to achieve its maximal historical interest and hilarity.

@HersonHN
Copy link

@HersonHN HersonHN commented Oct 29, 2013

Well, someone had to do this https://npmjs.org/package/dorkq

@Boldewyn
Copy link

@Boldewyn Boldewyn commented Oct 29, 2013

@andrewsf D’oh! Thanks for the reminder, that one should read bug reports thoroughly!

For the "freezing": I’d love to see one last follow-up comment of the glorious person, that files the upstream bug with the Android browser, sharing the bug URL.

@gorkamolero
Copy link

@gorkamolero gorkamolero commented Oct 30, 2013

This is crazy

@cowboy
Copy link
Member

@cowboy cowboy commented Nov 2, 2013

LOL I just saw this

@jpdevries
Copy link

@jpdevries jpdevries commented Nov 3, 2013

Please make a tshirt parody of this bug. It would sell like crazy

@gorkamolero
Copy link

@gorkamolero gorkamolero commented Nov 3, 2013

I second the T-shirt idea

@dangribbin
Copy link

@dangribbin dangribbin commented Jan 27, 2014

Just saw this, but I will buy a dorkq shirt for sure.

@jmaxxz
Copy link

@jmaxxz jmaxxz commented Feb 3, 2014

👍 For the tshirt

@Pomax
Copy link

@Pomax Pomax commented Jan 26, 2016

I'd seriously still like to know what happened here, because I know my OpenType, and unless the font somehow has GSUB rules specific to running a Caesar cypher, that is not a thing that will spontaneously happen.

@davelab6
Copy link

@davelab6 davelab6 commented Jan 27, 2016

There was a bug in Java where fonts were cached in way that used Glyph IDs, so when a font was updated and the GID order changed, you'd see a caesar cypher effect. Its possible there is such a bug in Android.

@Pomax
Copy link

@Pomax Pomax commented Jan 27, 2016

That's quite the bug. Would you happen to know the bug link for that on the java side?

@ashrocket
Copy link

@ashrocket ashrocket commented Jan 27, 2016

Tee spring is a YC company that does exactly this. https://teespring.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
You can’t perform that action at this time.