-
-
Notifications
You must be signed in to change notification settings - Fork 336
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Opt-In Typography Styles #1189
Comments
Noting Tailwind v3.3 feature for review: |
Note: font sizes should be built into theme settings, design tokens added |
FYI I've started a new PR here: #1374 I've made a few modifications so far:
Per number 5, here's a before and after: h1:not(.unstyled):is(:not(.prose *)) { ... } h1:not(.unstyled) { ... } Note that current updates do not address the primary issue yet - typography is not yet opt in. One of the challenges we're going to face is even when the default styles are used, folks still need to an escape hatch. So for now But I'll return to this tomorrow. I'm testing several ideas, with the goal minimizing regressions for existing users. If there's a migration process, I'll keep it as simple as possible. |
The new opt-in typography system is now available in the PR. This is a work in progress so not all changes have been fully implemented into our documentation yet. However, please read below to learn how this will work. Stylesheet UpdatesA number of updates have been made to stylesheets:
In short,
Opt-In Typography and MigrationTo notify and prepare users for the upcoming change, I've updated the documentation to assume the opt-in option as the default. Additionally I've introduced a detailed explanation in the warning box at the top of the Typography docs: https://skeleton-docs-git-feat-opt-in-typography-skeleton-labs.vercel.app/elements/typography This explains what is happening, what the benefit is, as well as how to actually migrate. From v1.3+ we should assume new users are using the opt-in approach, while legacy users should be encourage to migrate asap. When v2.x drops the opt-in option will be the only available, which represents a breaking change. So updating sooner is in everyone's best interest. Updating the Skeleton DocsFor now I've opted to wait until users can provide feedback on this change. We're using the |
@SebasF1349 made a great suggestion on Discord, we'll leave the legacy I'll aim to implement this change tomorrow. Thanks Sebas! UPDATE This change has bee implemented into the PR |
FYI I've completed and merged this change. We'll provide a full announcement and recommendation for this in the next release notes. For now I'm going to create another PR that begins the process of updating our documentation site to utilize the new opt-in styles. This may be an on-going effort given the scale and scope of our doc site. You can preview the upcoming changes here: |
Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
We've had a lot of folks struggling with Typography styles overrides, either locally or globally. Let's look into a means make it easier to opt-in. We found a lot of success with inputs, so let's follow the lead there.
What type of pull request would this be?
Enhancement
Any links to similar examples or other references we should review?
No response
The text was updated successfully, but these errors were encountered: