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

[Display Layout] Lines can be edited when not in edit mode. #6846

Closed
2 of 7 tasks
rukmini-bose opened this issue Jul 27, 2023 · 5 comments · Fixed by #6858
Closed
2 of 7 tasks

[Display Layout] Lines can be edited when not in edit mode. #6846

rukmini-bose opened this issue Jul 27, 2023 · 5 comments · Fixed by #6858
Assignees
Labels
bug:regression It used to work. Now it doesn't :( severity:critical type:bug
Milestone

Comments

@rukmini-bose
Copy link
Contributor

rukmini-bose commented Jul 27, 2023

Summary

In Display Layouts, you can currently edit a line's shape and size when NOT in edit mode. This should not be possible– if you aren't in edit mode, no draw objects should be able to be changed. This is potentially a big issue since it could allow users to accidentally drag lines when viewing, and potentially mess up complex displays. In addition, the change persists so upon refresh so there's no way to go back to the previous settings of the layout.

Untitled.mov

Expected vs Current Behavior

Lines should not be able to be adjusted when not in edit mode.

Steps to Reproduce

  1. Create a/navigate to a Display Layout
  2. In edit mode, add a line object to the layout.
  3. Save the settings.
  4. Click the line, and you can see that you can drag the line, adjust its length and orientation.

Environment

  • Open MCT Version: Version: 3.0.0-SNAPSHOT
  • 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 ozyx added this to the Target:3.0.0 milestone Jul 27, 2023
@ozyx ozyx added severity:critical bug:regression It used to work. Now it doesn't :( labels Jul 27, 2023
@shefalijoshi
Copy link
Contributor

It appears that this has been the behavior for a while:
https://github.com/nasa/openmct/blob/master/src/plugins/displayLayout/components/DisplayLayout.vue#L199

@rukmini-bose
Copy link
Contributor Author

Verified Testathon 7/31/2023

@khalidadil
Copy link
Contributor

Verified Fixed in Testathon on 07/31/23

@ozyx
Copy link
Member

ozyx commented Jul 31, 2023

Verified Fixed Testathon 07/31/23

@charlesh88
Copy link
Contributor

Testathon 2023-07-31: verified fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:regression It used to work. Now it doesn't :( severity:critical type:bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants