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

Make it possible to use style sheet URLs again #870

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

mattiasb
Copy link
Contributor

@mattiasb mattiasb commented Jan 8, 2025

Only expand markdown-css-paths that clearly are relative.

Description

This disables the unreleased support for relative paths without a ./, ../ or ~ prefix.

Add an extra markdown-css-paths test that ensures we don't break URL support going forward.

Related Issue

Fixes #869.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves an existing feature)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the CONTRIBUTING.md document.
  • I have updated the documentation in the README.md file if necessary.
  • I have added an entry to CHANGES.md.
  • I have added tests to cover my changes.
  • All new and existing tests passed (using make test).

Only expand `markdown-css-paths` that clearly are relative.

This disables the *unreleased* support for relative paths without a
"./", "../" or "~" prefix.

Add an extra `markdown-css-paths` test that ensures we don't break URL
support going forward.

Fixes jrblevin#869.
@syohex syohex merged commit e100778 into jrblevin:master Jan 9, 2025
4 checks passed
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.

markdown-css-paths is broken
2 participants