Skip to content

Allow to import such utils as round, rem, em #302

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

Closed
wants to merge 1 commit into from

Conversation

talaxasy
Copy link

When a person wants to extend the typography config not only colors, e.g.:

extend: {
      typography: ({ theme }) => ({
        lg: {
          css: {
            h1: {
              fontSize: em(38, 18),
              marginTop: '0',
              marginBottom: em(40, 48),
              lineHeight: round(48 / 48),
            },
          },
        },

When a person wants to extend the typography config not only colors, e.g.:

```
extend: {
      typography: ({ theme }) => ({
        lg: {
          css: {
            h1: {
              fontSize: em(38, 18),
              marginTop: '0',
              marginBottom: em(40, 48),
              lineHeight: round(48 / 48),
            },
          },
        },
```
@vercel
Copy link

vercel bot commented Mar 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
tailwindcss-typography ❌ Failed (Inspect) Mar 28, 2023 at 11:31AM (UTC)

@thecrypticace
Copy link
Contributor

Hey! While we understand the usefulness of exporting these helpers we don't currently want to increase the API surface area of the package. They're really meant as internal helpers only and would encourage you to copy them to your project if you feel the need to use them — as we could change them or our use of them at any time. Appreciate the PR though! ✨

p.s. The export … syntax isn't valid here as the file is a Common JS module. :)

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.

2 participants