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

Expand font-size and font-weight scales, rename classes for clarity, fix docs #636

Open
wants to merge 9 commits into
base: v5.0.0-1
Choose a base branch
from

Conversation

jeremykohn
Copy link

Expand font-size type scale to ten sizes, including 4rem as well as all the sizes from Tachyons v4. Rename font-size classes to use fs prefix: fs1-fs10, with larger numbers for larger font sizes. Largest class fs10 has alias fs-headline, and next largest class fs9 has alias fs-subheadline.

Rename fs-normal class to fstyle-normal, so class naming is consistent: fs prefix for font-size, and fstyle prefix for font-style.

New class names are all different from Tachyons v4 class names, to avoid conflicting with legacy code.

Expand `font-size` type scale to ten sizes, including `4rem` as well as all the sizes from Tachyons v4. Rename `font-size` classes to use `fs` prefix: `fs1`-`fs10`, with larger numbers for larger font sizes. Largest class `fs10` has alias `fs-headline`, and next largest class `fs9` has alias `fs-subheadline`.

Rename `fs-normal` class to `fstyle-normal`, so class naming is consistent: `fs` prefix for `font-size`, and `fstyle` prefix for `font-style`.

New class names are all different from Tachyons v4 class names, to avoid conflicting with legacy code.
@jeremykohn jeremykohn mentioned this pull request Aug 20, 2019
17 tasks
Add .fstyle-i class for font-style: italic, and use .i as an alias for fstyle-i. Update docs accordingly: specify that i, fs-headline, and fs-subheadline are aliases for other classes.
Add class fw-normal for font-weight: normal. Add class fw-b for font-weight: bold, and use class b as an alias for that class. For literal values of font-weight, rename classes fw1 through fw9 to fw-100 through fw-900, because Tachyons class names for literal values usually follow that pattern. Add class fw-1000 for font-weight: 1000. Update docs accordingly.
@jeremykohn
Copy link
Author

jeremykohn commented Aug 21, 2019

Create classes fstyle-italic for font-style: italic, and fw-bold for font-weight: bold. Use i and b as aliases for those classes. Add "Aliases" headings to docs to specify which classes are aliases.

Add class for font-weight: 1000 which is the maximum font weight value. Re-add class for font-weight: normal. Rename font-weight classes for clarity and to be consistent with how literal classes are usually named in Tachyons.

In the docstrings, the headings "Literal Values" really mean "Modifiers," so rename them as such.

Update docs/class-naming.md to reflect the new class names.

Already updated font-style and font-weight classes. Now make the corresponding updates to small, medium, and large font-style and font-weight classes.
@jeremykohn jeremykohn changed the title Expand font-size scale, rename classes for clarity Expand font-size scale, rename classes for clarity, fix docs Aug 21, 2019
@jeremykohn jeremykohn changed the title Expand font-size scale, rename classes for clarity, fix docs Expand font-size and font-weight scales, rename classes for clarity, fix docs Aug 21, 2019
Update docs to match the changes in CSS code for font-weight, font-size, and font-style classes.
@jeremykohn jeremykohn mentioned this pull request Aug 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant