Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

Preload the font used by .keyboard-focus #26

Closed
wants to merge 1 commit into from

Conversation

blasten
Copy link
Contributor

@blasten blasten commented Jun 18, 2015

If a button receives focus from the keyboard, the font-face might not be ready (downloaded & decoded). This creates an undesirable visual effect; however, if the font-face is requested ahead in time, we can avoid that effect.

This seems to be the issue in #24
\cc @frankiefu

@@ -120,6 +120,12 @@
.content {
padding: 0.7em 0.57em
}

/* preload the font used by `.keyboard-focus` */
::after {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Such hack.

@cdata
Copy link
Contributor

cdata commented Jul 13, 2015

Does this actually work?

Also, maybe you should run the tests again?

Also also, sorry for responding to this so late.

Finally 👍

@cdata
Copy link
Contributor

cdata commented Oct 5, 2015

@blasten Can you rebase this branch, and then reopen the PR? Thanks.

@cdata cdata closed this Oct 5, 2015
@notwaldorf notwaldorf deleted the keyboard-focus-font branch November 12, 2015 20:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants