🚧 WIP - Fix TinaCMS issue 6300 #6301
Conversation
Added test coverage for video link timestamp preservation to ensure query parameters (t=, start=) and hash fragments are correctly maintained in sanitized URLs. Tests cover: - YouTube standard format (/watch?v=...&t=...) - YouTube short format (youtu.be/...?t=...) - YouTube start parameter (&start=...) - Vimeo hash timestamps (#t=...s) Fixes #6300
|
Playwright E2E Test Results
Details
Skipped testschromium › retain-input-value-from-localstorage.spec.ts › Local storage retain edit test |
|
@kulesy For embed link it is |
Extended test coverage to include YouTube embed URLs (used in iframes) which use the /embed/ path format and different timestamp parameters: - YouTube embed with start parameter (?start=60) - YouTube embed with multiple parameters (?start=60&autoplay=1) These URLs are commonly used when embedding videos in iframes and need to preserve their query parameters to maintain timestamp functionality. Related to #6300
Created comprehensive round-trip test (markdown → Plate → markdown) to verify that video URLs with timestamp parameters are correctly preserved throughout the entire editing flow. Test cases cover: - YouTube watch URLs with &t= parameter - YouTube short URLs (youtu.be) with ?t= parameter - YouTube embed URLs with ?start= parameter (for iframes) - Vimeo URLs with #t= hash parameter This test verifies that when a user: 1. Opens markdown with video links in the TinaCMS editor 2. Edits content in the rich text editor 3. Saves back to markdown The video timestamp parameters are preserved correctly, ensuring videos start at the specified time when links are clicked. Fixes #6300
Added test coverage for video link timestamp preservation to ensure query parameters (t=, start=) and hash fragments are correctly maintained in sanitized URLs. Tests cover:
Fixes #6300