multiply legend_font_pointsize by line_width

[t-bltg]

Fix #3003.

The inconsistency should be fixed.

Needs new reference images. Maybe we need some non-linearity or flag here to revert to non-scaling of legend line width. TBD.

PR

master
https://docs.juliaplots.org/dev/gallery/gr/generated/gr-ref1/#gr_demo_1

[BeastyBlacksmith]

I mean.. I don't like that scaling at all, but being consistent is better than not

[t-bltg]

Maybe multiply by min(lw, 5), so that it doesn't get too large in the legend ?
🤔

[t-bltg]

I've arbitrarily clamped the legend linewidth in the range [0.2 - 5] (default is lw=1) so that legend boxes with lw outside this range still look acceptable.

It think clamping is required, since we want to avoid legend entries overlap.

What are your thoughts on these values @BeastyBlacksmith ?

using Plots
x = rand(10)
plot(x, lw=0.1)
plot(x, lw=5)
plot(x, lw=10)

lw=0.1

lw=5

lw=10

[BeastyBlacksmith]

Looks reasonable 👍🏻

[codecov]

Codecov Report

Base: 91.08% // Head: 91.09% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (994c44a) compared to base (5fd4d60).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4530   +/-   ##
=======================================
  Coverage   91.08%   91.09%           
=======================================
  Files          40       40           
  Lines        7683     7689    +6     
=======================================
+ Hits         6998     7004    +6     
  Misses        685      685           

Impacted Files	Coverage Δ
src/args.jl	84.70% <100.00%> (ø)
src/backends/gr.jl	92.71% <100.00%> (+0.03%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[syou83syou83]

Can anybody share an example code how this problem has been solved? I don't quite get it. Thank you very much. @t-bltg @BeastyBlacksmith