Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

add font-display property #12785

Merged
merged 1 commit into from
Sep 17, 2016
Merged

add font-display property #12785

merged 1 commit into from
Sep 17, 2016

Conversation

samuelcarreira
Copy link
Contributor

Docs: https://developer.mozilla.org/en-US/docs/Web/CSS/@font-face/font-display

****** Values **********
auto
The font display strategy is defined by the user agent.
block
Gives the font face a short block period and an infinite swap period.
swap
Gives the font face no block period and an infinite swap period.
fallback
Gives the font face an extremely small block period and a short swap period.
optional
gives the font face an extremely small block period and no swap period.

Docs: https://developer.mozilla.org/en-US/docs/Web/CSS/@font-face/font-display

****** Values **********
auto
The font display strategy is defined by the user agent.
block
Gives the font face a short block period and an infinite swap period.
swap
Gives the font face no block period and an infinite swap period.
fallback
Gives the font face an extremely small block period and a short swap period.
optional
gives the font face an extremely small block period and no swap period.
@zaggino
Copy link
Contributor

zaggino commented Sep 16, 2016

@samuelcarreira You need to sign http://dev.brackets.io/brackets-contributor-license-agreement.html before I can merge this in.

@zaggino zaggino added this to the Release 1.9 milestone Sep 16, 2016
Copy link
Collaborator

@ficristo ficristo left a comment

Choose a reason for hiding this comment

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

I think you should sort the values alphabetically.
It seems to me that only inherit should be always the last value

@ficristo
Copy link
Collaborator

I should have read all the description...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants