🧪 (line legend) add tests for dropping labels

[sophiamersmann]

Adds tests in preparation for a refactor of the code responsible for dropping labels when there is not enough space.

I also renamed some of the computed values in the line legend component for clarity.

The SVG tester comes back with a difference because there was a small issue about label padding that I fixed.

[sophiamersmann]

• 🔨 (line legend) refactor label dropping algorithm #4310
• 🧪 (line legend) add tests for dropping labels #4309 👈 (View in Graphite)
• 🔨 add empty arrays to the default grapher config #4294 : 1 other dependent PR (#4243 )
• ✨ (slope) improve svg output for figma #4292
• 🎉 (line+slope) add focus state / TAS-739 #4272
• 🔨 move colors to ColorConstants file #4289
• ✨ improve hover state of facet legends #4285
• 🔨 refactor hover states / TAS-738 #4266
• 🎉 make line/slope switching available to authors / TAS-720 #4207 : 1 other dependent PR (#4210 )
• 🎉 (slope) support facetting / TAS-734 #4247
• ✨ (slope) redesign / TAS-722 #4222
• 🎉 (slope) add tolerance / TAS-719 #4205
• 🎉 (slope) feature parity with line charts / TAS-708 #4192
• 🔨 migrate slope chart configs #4275
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[owidbot]

Quick links (staging server):

Site Dev	Site Preview	Admin	Wizard	Docs

Login: ssh owid@staging-site-line-legend-tests-viz

• Site-screenshots: https://github.com/owid/site-screenshots/compare/line-legend-tests-viz
• SVG tester: https://github.com/owid/owid-grapher-svgs/compare/line-legend-tests-viz

SVG tester:

Number of differences (default views): 2 (763b68) ❌
Number of differences (all views): 0 ✅

Edited: 2024-12-19 10:39:16 UTC
Execution time: 1.47 seconds

[sophiamersmann]

Merge activity

• Dec 19, 7:11 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Dec 19, 7:12 AM EST: A user merged this pull request with Graphite.