When transforming index.md urls within a content collection, the transforming is inconsistent between index.md in the collection root and index.md in collection subdirectories.
For example, ./src/content/docs/index.md with a custom slug of '' will transform to /docs/ (includes /) whereas ./src/content/docs/child/index.md will transform to /docs/child (doesn't include /).
In PR #20, trailingSlash support was added and it forced a '/' when the custom slug is empty string (''). This behavior is correct when the collection is acting as the site root (collectionPathMode='root') but when the collection is not in the site root, a trailingSlash should not be forced and defer to the trailingSlash setting.
Click on Docs Index link to enter into the collection index page
Hover over (or inspect in dev tools) the links generated for Docs Index, Docs Child Index and Docs Grandchild Index
Expected Result:
The same behavior should be applied to all links - none of them should end in trailing slash
Docs Index => /docs
Docs Child Index = > /docs/child
Docs Grandchild Index =>/docs/child/grandchild`
Actual Result:
The behavior is different for Docs Index which ends in a trailing slash compared to Docs Child Index & Docs Grandchild Index which do not end in a trailing slash
Docs Index => /docs/
Docs Child Index = > /docs/child
Docs Grandchild Index =>/docs/child/grandchild`
Additional Information
The current behavior of forcing a forward slash / in the generated URL for the collection root itself should still remain when the collectionPathMode='root' (e.g., the docs/index.md url should transform to / and for docs/child/index.md and docs/child/grandchild/index.md there should be no trailing slash, /child and /child/grandchild respectively).
When transforming
index.mdurls within a content collection, the transforming is inconsistent betweenindex.mdin the collection root andindex.mdin collection subdirectories.For example,
./src/content/docs/index.mdwith a custom slug of''will transform to/docs/(includes/) whereas./src/content/docs/child/index.mdwill transform to/docs/child(doesn't include/).In PR #20,
trailingSlashsupport was added and it forced a'/'when the custom slug is empty string (''). This behavior is correct when the collection is acting as the site root (collectionPathMode='root') but when the collection is not in the site root, a trailingSlash should not be forced and defer to thetrailingSlashsetting.Repro: https://stackblitz.com/edit/withastro-astro-l1egoq
Steps to reproduce:
Docs Indexlink to enter into the collection index pageDocs Index,Docs Child IndexandDocs Grandchild IndexExpected Result: The same behavior should be applied to all links - none of them should end in trailing slash
Docs Index=>/docsDocs Child Index= >/docs/childDocs Grandchild Index =>/docs/child/grandchild`Actual Result: The behavior is different for
Docs Indexwhich ends in a trailing slash compared toDocs Child Index&Docs Grandchild Indexwhich do not end in a trailing slashDocs Index=>/docs/Docs Child Index= >/docs/childDocs Grandchild Index =>/docs/child/grandchild`Additional Information
The current behavior of forcing a forward slash
/in the generated URL for the collection root itself should still remain when thecollectionPathMode='root'(e.g., thedocs/index.mdurl should transform to/and fordocs/child/index.mdanddocs/child/grandchild/index.mdthere should be no trailing slash,/childand/child/grandchildrespectively).