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

Add the option to enable/disable chart data points #69

Closed
TheRustyPickle opened this issue Feb 27, 2024 · 0 comments · Fixed by #83
Closed

Add the option to enable/disable chart data points #69

TheRustyPickle opened this issue Feb 27, 2024 · 0 comments · Fixed by #83
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@TheRustyPickle
Copy link
Owner

TheRustyPickle commented Feb 27, 2024

The chart renders all the tx method data at once. Perhaps another row can be added with all the tx method names where they can be enabled or disabled on perhaps Space or Enter press.

  • Add a new row that will show all tx method
  • The new row will be selectable/cyclable similar to how the other rows are
  • Default all methods will be selected
  • Selected methods should have a different color
  • Will become hidden on double 'r' press as it happens on the chart page right now
  • Selected methods should be saved in a vector(?) or something similar and passed to the chart UI where it will render based on that. Should ignore all data points if not selected

If anyone wants to contribute, feel free to comment here! Partial contribution toward this will be accepted. Will provide a guide and info as necessary.

@TheRustyPickle TheRustyPickle added enhancement New feature or request good first issue Good for newcomers labels Feb 27, 2024
@TheRustyPickle TheRustyPickle self-assigned this Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant