Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 hide entity name when a single entity is selected in multi-indicator line and slope charts #4601

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sophiamersmann
Copy link
Member

@sophiamersmann sophiamersmann commented Feb 26, 2025

Fixes #4600

I agree with Pablo R that not showing the entity name is the better default if a single entity is selected (regardless of the entity selection setting).

I can see in the code that showing the entity name in such cases was a deliberate choice, but I can't think of a good reason.

We persist the series name in the URL, so ideally the series name of a line shouldn't change throughout a session. Working on a solution...

@github-actions github-actions bot added the staging-viz Let SVG tester fail silently in CI label Feb 26, 2025
@sophiamersmann sophiamersmann force-pushed the line-chart-label-entity-name branch from 344cc8f to 35a6654 Compare February 26, 2025 17:00
@owidbot
Copy link
Contributor

owidbot commented Feb 26, 2025

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-line-chart-label-entity-name

SVG tester:

Number of differences (default views): 5 (197a1e) ❌
Number of differences (all views): 0 ✅

Edited: 2025-02-26 17:18:39 UTC
Execution time: 1.21 seconds

@sophiamersmann sophiamersmann marked this pull request as ready for review February 26, 2025 17:19
@sophiamersmann sophiamersmann requested review from marcelgerber and removed request for marcelgerber February 26, 2025 17:19
@sophiamersmann sophiamersmann marked this pull request as draft February 27, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
staging-viz Let SVG tester fail silently in CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Entity names should be hidden on line charts with multiple indicators when only one entity is chosen
2 participants