8000 Reset border styles in Tailwind compatibility package by HiDeoo · Pull Request #3132 · withastro/starlight · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Reset border styles in Tailwind compatibility package #3132

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

Merged
merged 2 commits into from
Apr 17, 2025

Conversation

HiDeoo
Copy link
Member
@HiDeoo HiDeoo commented Apr 16, 2025

Description

The Starlight Tailwind compatibility package restores only some crucial styles from Tailwind Preflight instead of using it entirely.

This previous version of the Tailwind compatibility package used to restore a rule resetting all border styles to allow adding a border to an element by just adding a border-width.

The new version is no longer restoring this rule (which still exists in Tailwind v4) which could cause updating users to have default borders on their buttons. This PR adds it back to the compatibility package.

For example, this issue was initially reported while updating the Cloudflare Docs, here is how it looks with Tailwind v4 without the fix and with the fix:

Before After
SCR-20250417-issh SCR-20250417-itin

Copy link
changeset-bot bot commented Apr 16, 2025

🦋 Changeset detected

Latest commit: fe73945

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

This PR includes changesets to release 1 package
Name Type
@astrojs/starlight-tailwind Patch

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

@github-actions github-actions bot added the 🌟 tailwind Changes to Starlight’s Tailwind package label Apr 16, 2025
Copy link
netlify bot commented Apr 16, 2025

Deploy Preview for astro-starlight ready!

Name Link
🔨 Latest commit fe73945
🔍 Latest deploy log https://app.netlify.com/sites/astro-starlight/deploys/6800e4b1202f420008a3c8d9
😎 Deploy Preview https://deploy-preview-3132--astro-starlight.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@HiDeoo HiDeoo marked this pull request as draft April 16, 2025 16:38
@HiDeoo HiDeoo marked this pull request as ready for review April 17, 2025 07:46
@HiDeoo HiDeoo merged commit 5959919 into withastro:main Apr 17, 2025
15 checks passed
@astrobot-houston astrobot-houston mentioned this pull request Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌟 tailwind Changes to Starlight’s Tailwind package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0