assign pt.lwd for type "p" in legend

[kscott-1]

Fixes #319. Looking at the issue I posted earlier, it appears that the jitter plot is internally referenced as "p" type. While my knowledge of this codebase is next to nothing, I don't believe there should be unintended consequences of this change because "p" should only include points. The changes allow for the legend to pick up the lwd argument in the case that the pch used during plotting can be impacted by that value. Happy to hear more if I have misunderstood this :)

set.seed(0)
x <- rep(c("A", "B", "C"), each = 20)
y <- runif(60)
gp <- sample(c("X", "Y", "Z"), size = 60, replace = TRUE)
df <- data.frame(x, y, gp)

library(tinyplot)
utils::packageVersion("tinyplot")
#> [1] '0.3.0.99'
plt(
  y ~ x | gp,
  data = df,
  pch = 0:2,
  lwd = 4,
  type = type_jitter()
)

[grantmcdermott]

Thanks @kscott-1, good catch!

Do you mind adding/committing a short NEWS.md item under the "Bugs fixes" section? Please include your GH handle too, so that folks know who to credit.

I'll merge once that's done.

[kscott-1]

Thanks @grantmcdermott, let me know if that works for you. Glad I could contribute to a great project.

[vincentarelbundock]

This is very cool. Thanks for the bug fix!

[grantmcdermott]

Thanks!

[grantmcdermott]

(Your PR also reminded me of a separate issue related to handing of character variables and default types, which I'll address separately.)