Skip to content

Conversation

Hugos68
Copy link
Contributor

@Hugos68 Hugos68 commented Feb 1, 2025

Linked Issue

Closes #3195

Description

This is the rewrite of the Skeleton core package to support Tailwind v4, previously referred to as the Skeleton Tailwind plugin. It utilizes Vite for bundling and SCSS (aka Sass) for authoring our styles.

Overall Progress

High level overview of the entire scope of this pull request. When all items are green, this PR is ready for review.

Status Feature
Core Package
Component Packages
Theme Generator
Documentation
Migration Guides

Core Package Features

This represents progress specific to the core package.

Base

Status Feature
Theme
Globals
Colors
Color Pairings
Typography
-- Spacing

Themes

Status Feature
Skeleton Themes

Animations

Status Feature Method
Animations Extended

Utilities

Status Feature
Badges
Buttons
Cards
Chips
Dividers
Forms / Inputs
Tables
Placeholders
-- Gradients

Optional

Status Feature Method
Presets Added

Copy link

changeset-bot bot commented Feb 1, 2025

🦋 Changeset detected

Latest commit: da0009d

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Feb 1, 2025

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

Name Status Preview Comments Updated (UTC)
skeleton-docs ❌ Failed (Inspect) Feb 24, 2025 7:40pm
skeleton-next ❌ Failed (Inspect) Feb 24, 2025 7:40pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
skeleton-themes ⬜️ Ignored (Inspect) Visit Preview Feb 24, 2025 7:40pm

Copy link
Member

@AdrianGonz97 AdrianGonz97 left a comment

Choose a reason for hiding this comment

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

some more stuff

Comment on lines +84 to +87
/* .focused {
@apply ring-[2px] ring-surface-950 dark:ring-surface-50 ring-inset;
} */
}
Copy link
Member

Choose a reason for hiding this comment

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

should this still be commented out/hasn't been converted to standard css yet?

Copy link
Contributor

@endigo9740 endigo9740 Feb 24, 2025

Choose a reason for hiding this comment

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

Let's move this to the Zag 1.0 PR for consideration. This will be part of a large discussion around how Zag depends on CSS custom properties to define certain component values and styles. In this case the focus state outline.

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.

4 participants