antvis/Graphin

Graphin.Legend should hide node halo as well

Offen

#488 geöffnet am 12.06.2023

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (276 Forks)auto 404
help wanted

Repository-Metriken

Stars
 (1.096 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Describe the bug

If the node has a halo, when we click on the legend to "hide" that node, the halo is still fully visible but it should take the node visibility into account to make it also invisible, or less visible.

Your Example Website or App

https://codesandbox.io/s/suspicious-heisenberg-rhzvzc?file=/App.tsx

Steps to Reproduce the Bug or Issue

Add a legend to a chart, display nodes with halos and click the legend to toggle node visibility.

Expected behavior

The halo should also go invisible alongside the node.

Screenshots or Videos

No response

Platform

  • OS: Linux
  • Browser: Edge, Chrome
  • Version: LTS

Additional context

No response

Contributor Guide

Research-Richtung
Untersuchen Sie, wie das Umschalten der Legende Knoten ausblendet, und identifizieren Sie, warum der Halo sichtbar bleibt. Suchen Sie nach der Halo Rendering Logik im Quellcode von Graphin und stellen Sie sicher, dass diese den Sichtbarkeitszustand des Knotens respektiert.
Tech Stack
typescriptreact
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
React
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60