Skip to content
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

Update CSS, add vite ignore flags to dynamic imports #3

Merged
merged 2 commits into from
Jan 5, 2025

Conversation

Adammatthiesen
Copy link
Member

@Adammatthiesen Adammatthiesen commented Jan 5, 2025

This pull request includes several updates and improvements to the @studiocms/markdown-remark package, focusing on CSS changes, dynamic import handling, and enhancements to the Astro integration tests. The most important changes include adding flags to prevent Vite dynamic import warnings, updating CSS, and refactoring Astro integration tests to use a new layout component.

Dynamic Import Handling:

  • Added /* @vite-ignore */ flags to dynamic imports in import-plugin-browser.ts and import-plugin-default.ts to prevent Vite warnings. [1] [2]

CSS Updates:

  • Updated headings.css by removing unused media queries and CSS rules. [1] [2]

Astro Integration Tests:

  • Refactored multiple Astro integration test files to use a new Layout component for consistent HTML structure. [1] [2] [3] [4]
  • Added new test content to md.txt files to improve test coverage and ensure proper rendering.

Miscellaneous:

  • Updated package.json scripts to streamline the development process and ensure all relevant packages are included in the dev script.
  • Created a new layout.astro file to define the layout structure used across multiple test pages.

…e usable for dev instead of including a playground
@Adammatthiesen Adammatthiesen requested a review from a team as a code owner January 5, 2025 09:33
Copy link

pkg-pr-new bot commented Jan 5, 2025

Open in Stackblitz

npm i https://pkg.pr.new/@studiocms/markdown-remark@3

commit: 1ce31d8

@Adammatthiesen Adammatthiesen merged commit 74d2b32 into main Jan 5, 2025
4 checks passed
@Adammatthiesen Adammatthiesen deleted the fix-heading-wrapper branch January 5, 2025 10:18
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