DOCS: generate smaller tables for feature matrix#3579
DOCS: generate smaller tables for feature matrix#3579chicks-net wants to merge 21 commits intoStackExchange:mainfrom
Conversation
|
I don't have much of an opinion either way on keeping the scrolling table or breaking it up. Using three tables would solve the horizontal scrolling problem at least. As for grouping, maybe the RR types should be rearranged to keep related types (and meta-types) together. For instance, I think the data should definitely be included on each provider page as a supported features table with just two columns: feature and green/red/? status. |
😁
That sounds good. I will try that.
This was my original goal and I still think it is a great idea. I'm guessing that will be a different PR though. Fixing the existing table has helped me understand what I will need to do that. I will clean up this code and request more feedback once it is ready for a review. (It will probably be next week, since we're babysitting this weekend.) |
|
I've moved this out of draft - please review. Looking at the preview in GitBook is encouraging, but I'm guessing we should aim for 4 tables. I'll work on that next. |
|
I'm not sure this is better than the side-scrolling table, but it's not really worse either. It's just a lot of data to fit. I wish we had better control over the Gitbook layout and styles so something like sticky headers and columns or even column filtering could be used. I found an interesting hack using SVG to stuff css styles that works in GitHub markdown rendering. I didn't try it in Gitbook. If there are to be multiple tables, the 'Table 1', 'Table 2' section headings should be omitted or used to highlight which features are included in the tables (not sure how to best summarize that for a terse heading though). |
|
I'm working in https://gist.github.com/chicks-net/e3533d8a766013895f917dc4a22b7761 to categorize all of the columns and see if i can find a more logical organization. Suggestions are welcomed. |
|
New question: do any of the |
|
preview is looking good to me. 6 tables looks ok. The "DNS extensions" table has the most columns and the most word wrapping, but it is still pretty readable. Non-blocking issue: Does anybody know why GitBook doesn't handle Markdown paragraphs well? It was keeping the line endings from the file and not flowing the paragraph unless i had the whole paragraph on one line. GitBook says they support Commonmark which is pretty clear about how paragraphs in Markdown should work. |
|
yet another preview to document the dual-host column. For some reason it tries to link back to github - which 404s since this is not merged yet. This also effects the Concurrency Verified column on the live site. So it looks like a GitBook bug and I'm not sure why GitBook is acting this way. I'll see if I can fix this. If not, I'll make a new github issue for this problem. |
|
(Been a busy couple days... just catching up!) Wow! I saw the latest preview! Looks SO much better! The only suggestion I have so far is on the DNSSEC table... the AUTODNSSEC columns should be first. Can we make the table columns more narrow? (I'm guessing not, but I thought I'd ask) |
No worries! I wasn't expecting to be this far along before the weekend.
Thank you, thank you.
Agreed! Done in de66cf2 .
Reading https://gitbook.com/docs/creating-content/blocks/table makes it sound like this should be possible, but they don't explain how this works out at the Markdown level. I looked for other sites that use gitbook, but all of them that I've seen use basic table syntax. (I'm very new to GitBook, but I'm boggled by how much of their documentation is thin on the source code side. That's really not what I'd expect for something with So we're left with the old school techniques: removing characters from the titles, splitting up a title into multiple words, etc. I don't think the x/check/? cells are causing the columns to be wider. |
|
This is looking great @chicks-net ! I think the way you have categorized and broken up the tables makes it much easier to find the bit of information needed with less noise to ignore. |
Thanks for your comments, encouragement, and code. 🙏 It was fun to do some golang again after focusing on other languages recently. I also left https://gist.github.com/chicks-net/1b85b88851812075130bfc67d465bb3c out of this PR, but if anyone is interested in seeing how helpful |
|
LGTM! Please rebase and I'll merge. |
|
Gave up on fixing merge conflicts and opened #3589 . |
context
#3305 notes that the provider feature table does not display well in the GitBook-based documentation site.
This has also been brought up in a couple of the monthly live calls.
progress
This PR splits the features into different groups for different tables. It generates the tables with H3 headings. The changes to the generator and the documentation are included in this PR.
Some additions to
/.gitignoreare also included.future
Updating individual provider pages (#3584) is coming next.
Maybe file some bug reports with GitBook for the issues found.
prior "discussion" questions