docs icon indicating copy to clipboard operation
docs copied to clipboard

Search index missing New-PesterConfiguration codeblock

Open fflaten opened this issue 2 years ago • 8 comments

Checklist

  • [X] Issue has a meaningful title
  • [X] I have searched the existing issues. See issues

Link to page

https://pester.dev/docs/commands/New-PesterConfiguration

What is the issue?

Searching for SkipRemainingOnFailure returns no result even though it's documented in the New-PesterConfiguration help. I assume it's because codeblocks are excluded from indexing.

Suggested fix?

Is it possible to include codeblock for that specific page? Or should we focus on documenting all the options as part of other pages. Ex in this case as a section @ https://pester.dev/docs/usage/skip ?

fflaten avatar Sep 02 '23 14:09 fflaten

It looks like this Algolia thread contains a similar question. https://discourse.algolia.com/t/experience-and-advice-for-dealing-with-indexing-of-code-in-software-documentation/8307/3

It also mentions a specific tag that will help the DocSearch crawler to know what are the part important but I cannot seem to find an example. To me, that sentence reads as if it would be possible to include the codeblock only for that specific page.

bravo-kernel avatar Sep 04 '23 09:09 bravo-kernel

I'm considering generating this directly to markdown as a section in docs/Configuration maybe. Just like we do now in New-PesterConfiguration comment help.

Will fix this, have improved formatting and ToC for easier navigation.

Maybe also extract the generated help from New-PesterConfiguration to about_PesterConfiguration.txt in the module.

fflaten avatar Jun 09 '24 19:06 fflaten

Made a PoC. Thoughts?

List: image

Table with FQON (fully qualified option name 🤓): image

Table with option name: image

fflaten avatar Jun 13 '24 19:06 fflaten

Awesome, the FQON table is the most readable one for me. Super clear.

bravo-kernel avatar Jun 17 '24 07:06 bravo-kernel

Thanks. I'm undecided between FQON table and list. Table is bad on phone in portrait mode, while list use a lot of vertical space.

fflaten avatar Jun 17 '24 20:06 fflaten

How does the Tanstack table look bad on mobile? I use it myself and once I accepted the horizontal scrolbar it looks fine for me.

bravo-kernel avatar Jun 18 '24 06:06 bravo-kernel

How does the Tanstack table look bad on mobile? I use it myself and once I accepted the horizontal scrolbar it looks fine for me.

When I wrote it they overflowed and broke the layout. Fixed in https://github.com/pester/docs/commit/68bffbe0e80d75664b62bf3c60ff6416d8dbf358 with same wrapper used on tanstack tables earlier. 👷‍♂️ 👍

Personally not a fan of horizontal scrolling though.

fflaten avatar Jun 18 '24 15:06 fflaten

Yes, the overflow is a thing :)

bravo-kernel avatar Jun 18 '24 17:06 bravo-kernel