Skip to content

Docusaurus add tailwind #2965

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

Conversation

trillium
Copy link
Contributor

@trillium trillium commented Jun 8, 2025

Summary

This PR adds the same version of Tailwind CSS as the cursorless-org package (3.4.14).

Changes

  • Add Tailwind CSS configuration and dependencies to the org site.

Regression testing

Used Backstop.js to compare main branch with docusaurus-add-tailwind branch.

Passing: 65
Failing: 3

All failing tests have been the test suite not working quite right, screenshots below.

Backstop screenshot

Screenshots of test suite

image image

Screenshots of the urls of failing tests

image

image

image

@trillium trillium requested a review from a team as a code owner June 8, 2025 18:33
Copy link
Member

@pokey pokey left a comment

Choose a reason for hiding this comment

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

Looks fine with minor comment

@AndreasArvidsson
Copy link
Member

The lock file is not updated. Please run pnpm install and commit the updated lock file.

@trillium
Copy link
Contributor Author

trillium commented Jun 9, 2025

The lock file is not updated. Please run pnpm install and commit the updated lock file.

Removed!

Same small differences as before, screenshot attached.

Backstop Screenshot

image

@AndreasArvidsson AndreasArvidsson added this pull request to the merge queue Jun 9, 2025
Merged via the queue into cursorless-dev:main with commit 5aed28f Jun 9, 2025
15 checks passed
@trillium trillium deleted the docusaurus-add-tailwind branch June 9, 2025 17:27
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.

3 participants