docs icon indicating copy to clipboard operation
docs copied to clipboard

In content shortcuts anchors broken

Open damsfx opened this issue 1 year ago • 7 comments

On some docs pages, in text columned-list shortcuts don't work.
The three pages where I find the issue for now :

https://wintercms.com/docs/v1.2/docs/services/helpers top shortcuts https://wintercms.com/docs/v1.2/docs/services/validation#available-validation-rules https://wintercms.com/docs/v1.2/docs/services/collections#available-methods

For the Helpers top shortcuts, anchors need to be updated to remove "_".

For Validation (available validation rules) and Collections (available methods), the corresponding anchors doesn't exists in the rendered page.

damsfx avatar Oct 13 '23 14:10 damsfx

Can we just manually update them for now on the develop branch?

LukeTowers avatar Oct 14 '23 18:10 LukeTowers

@LukeTowers Is somebody working on this? This drive me crazy ...

damsfx avatar Jun 04 '24 14:06 damsfx

@bennothommo ping

LukeTowers avatar Jun 07 '24 04:06 LukeTowers

@damsfx the helpers page appears to be fixed now, do you want to submit a PR for the other ones?

LukeTowers avatar Jul 31 '24 08:07 LukeTowers

@LukeTowers I was going to fix the validation and collection ones at the same time, the problem though was that anchoring the rules/commands in those pages results in every validation rule / collection method appearing in the Table of Contents (anchors must be a H2 or H3). I wasn't sure if that was gonna be acceptable or not, so I left it.

bennothommo avatar Jul 31 '24 12:07 bennothommo

@LukeTowers I wanted to do it ... but I'm having trouble setting up the documentation locally to test my changes. Can you or @bennothommo give me some pointers on how to get a working copy of the local documentation? I alaways fall back in a Failed to open directory: No such file or directory error.

@bennothommo I had the same thought as you when I looked at the documentation plugin code. I even wondered whether it wouldn't be preferable for these pages to have their own title anchors in a H4 heading rather than polluting the TOC. But in this case it result in a markdown content having HTML.

damsfx avatar Jul 31 '24 13:07 damsfx

@bennothommo @damsfx I'm fine with the TOC being made bigger on those pages, eventually we'll reorganize the docs to have more pages with less content on each page so it's easier to find information by browsing around, but for now having them in the TOC should be fine. If necessary it would be nice if they could be collapsible, but that's not a hard requirement.

LukeTowers avatar Jul 31 '24 21:07 LukeTowers