Skip to content

Types vignette: list tinyplot types in neater table format#340

Merged
grantmcdermott merged 5 commits intomainfrom
types-tables
Mar 17, 2025
Merged

Types vignette: list tinyplot types in neater table format#340
grantmcdermott merged 5 commits intomainfrom
types-tables

Conversation

@grantmcdermott
Copy link
Copy Markdown
Owner

Purely aesthetic website change from the existing bullet points (which feel a bit crowded to me).

Thoughts? We can merge if other folks think that it makes for an improved reading experience.

Screenshot:

types-tables

@zeileis
Copy link
Copy Markdown
Collaborator

zeileis commented Mar 17, 2025

Very nice, I like it much better!

Small comment: Given that the descriptions are in sentence style, I would also use sentence case for the header row: String, Function, Description, Docs.

@grantmcdermott
Copy link
Copy Markdown
Owner Author

Thanks @zeileis. I actually tried this at first, but it felt a little too formal/distracting for my tastes. (I think b/c we already have quite a bit of nesting i.t.o header hierarchy, so adding another layer gets in the way of the content; if that makes sense?)

I need to check that the cross-references to the individual manual pages are working properly—I know that Quarto can be a bit finnicky about relative paths—and then will merge.

@grantmcdermott
Copy link
Copy Markdown
Owner Author

Minor aside: I was going to put "barplot" under "Shapes", but switched to "Visualizations" b/c it sometimes does automatic aggregation. LMK if you disagree.

@zeileis
Copy link
Copy Markdown
Collaborator

zeileis commented Mar 17, 2025

Another layer: I was not suggesting to add a layer - but just to say "String" instead of "string" etc.

Barplot as visualization: I agree, slightly better.

@grantmcdermott
Copy link
Copy Markdown
Owner Author

Another layer: I was not suggesting to add a layer - but just to say "String" instead of "string" etc.

Sorry, my reply was confusing (rushing before work). I get your suggestion, but I tried sentence case at first and it felt a bit too formal/heavy some reason. Maybe I'll feel differently in a couple weeks, but for I'll use my veto option to retain lowercase headings :-)

- adding leading "/" causes anchor relative to project root.
@grantmcdermott grantmcdermott merged commit 5166bc1 into main Mar 17, 2025
3 checks passed
@grantmcdermott grantmcdermott deleted the types-tables branch March 17, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants