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

[Overlay Plots] Removing 1 Object from the Elements Pool Removes 2 from the Legend #7527

Closed
1 of 7 tasks
rukmini-bose opened this issue Feb 27, 2024 · 4 comments · Fixed by #7531
Closed
1 of 7 tasks
Assignees
Labels
severity:critical type:bug verified Tested or intentionally closed
Milestone

Comments

@rukmini-bose
Copy link
Contributor

rukmini-bose commented Feb 27, 2024

Summary

In an Overlay Plot, removing an object from the elements pool is removing two objects from the legend. The two are being removed are the selected object and another random one. Sometimes, refreshing can bring the legend back to the correct state.

Look at this image which was taken after removing an element. There are 7 elements in the legend, while there are 8 in the elements pool.
Screenshot 2024-02-27 at 2 42 18 PM

Expected vs Current Behavior

When you remove an object from the Elements Pool, we should see only that object being removed from the plot and the legend.

Steps to Reproduce

  1. Navigate to an Overlay Plot that is populated with multiple data points.
  2. Count the number of elements in the plot.
  3. Go to Edit, and remove an object from the Elements Pool.
  4. Now go to the legend, and count the number of elements that are in it.
  5. Observe that there are 2 less elements in there, instead of 1.

Environment

  • Open MCT Version: 4.0.0-next
  • Deployment Type: testathon
  • OS:
  • Browser: Chrome

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

@ozyx
Copy link
Member

ozyx commented Feb 28, 2024

introduced in #7453

@ozyx
Copy link
Member

ozyx commented Mar 5, 2024

Testing Instructions

  • Follow steps to reproduce
  • Verify that removing 1 plot element from the tree context menu removes 1 legend item
  • Verify that removing 1 plot element from the elements pool context menu removes 1 legend item

@rukmini-bose
Copy link
Contributor Author

Verified Testathon 3/5/2024 YIPPEE

@jvigliotta
Copy link
Contributor

Verified Testathon 3/5/2024 BOOM

@ozyx ozyx added the verified Tested or intentionally closed label Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity:critical type:bug verified Tested or intentionally closed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants