Skip to content

Docusaurus add tailwind#2965

Merged
AndreasArvidsson merged 12 commits into
cursorless-dev:mainfrom
trillium:docusaurus-add-tailwind
Jun 9, 2025
Merged

Docusaurus add tailwind#2965
AndreasArvidsson merged 12 commits into
cursorless-dev:mainfrom
trillium:docusaurus-add-tailwind

Conversation

@trillium

@trillium trillium commented Jun 8, 2025

Copy link
Copy Markdown
Contributor

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

@pokey pokey left a comment

Copy link
Copy Markdown
Member

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

Comment thread packages/cursorless-org-docs/tailwind.config.js
Comment thread packages/cursorless-org-docs/tailwind.config.js
@AndreasArvidsson

Copy link
Copy Markdown
Member

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

@trillium

trillium commented Jun 9, 2025

Copy link
Copy Markdown
Contributor Author

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