Mistake in `type_sanitize`?

[grantmcdermott]

In the below code chunk, shouldn't "lines" be "l"? And the same for "points" -> "p"?

tinyplot/R/sanitize.R

Lines 49 to 53 in 9a56785

	"lines" = type_lines,
	"lm" = type_lm,
	"loess" = type_loess,
	"pointrange" = type_pointrange,
	"points" = type_points,

@vincentarelbundock I believe section goes back to you in ec4962b#diff-d6fbdd17c7d31c58cbbb80fa0e5be1b555404a61b03ad35e99d6e8cfecec1523

Am I missing something obvious about why these should be "points" and "lines" rather than "l" and "p"?

[vincentarelbundock]

I don't think this matters because the defaults for "l" are the same as the defaults for type_line(). We could add all the single letters as new lines in the switch for completeness, but I don't expect it to have any effect at all, so it's a bit of a waste.

We do need all non-Base string types there, though, so it's important to keep things like "jitter"

[grantmcdermott]

We could add all the single letters as new lines in the switch for completeness, but I don't expect it to have any effect at all, so it's a bit of a waste.

I think I might have caught a case where it does matter in #323 (comment) (see the final non-plot). I have to run to dinner now, but can follow up later.

[grantmcdermott]

Closed with #323