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

Domain object properties Notes don't appear in Inspector, but should #4430

Closed
2 of 5 tasks
charlesh88 opened this issue Nov 2, 2021 · 4 comments · Fixed by #4520
Closed
2 of 5 tasks

Domain object properties Notes don't appear in Inspector, but should #4430

charlesh88 opened this issue Nov 2, 2021 · 4 comments · Fixed by #4520
Labels
bug:regression It used to work. Now it doesn't :( severity:critical type:bug
Milestone

Comments

@charlesh88
Copy link
Contributor

charlesh88 commented Nov 2, 2021

Summary

This is raised in importance to solve a functional ask to allow users to add authorship and a version number to composed displays in Open MCT.

Expected vs Current Behavior

This used to function, filing this as a regression bug.

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?

Steps to Reproduce

  1. Edit properties on any composed view and add any content in the Notes field. Save and exit that dialog.
  2. Observe: the Notes content is not shown in the Inspector, but should be.

Environment

  • Open MCT Version: 1.7.8
  • Deployment Type: any
  • OS: any
  • Browser: any

Additional Information

image

image

@charlesh88 charlesh88 added type:bug bug:regression It used to work. Now it doesn't :( labels Nov 2, 2021
@khalidadil
Copy link
Contributor

@charlesh88 I talked to @davetsay and he didn't see notes in the inspector in a really old branch (before conditionals were implemented). Where should they be showing up in the inspector? Under their own heading? Below title and type? We can add them in based on the order you think they need to be in the inspector.

@khalidadil khalidadil assigned khalidadil and unassigned akhenry Nov 30, 2021
charlesh88 added a commit that referenced this issue Dec 7, 2021
- Removed word wrapping in Inspector value elements;
@unlikelyzero unlikelyzero added this to the Target: 1.8.2 milestone Dec 13, 2021
@jvigliotta
Copy link
Contributor

Verified Fixed - Testathon: 12/13/2021

@charlesh88
Copy link
Contributor Author

Testathon 12-13-21: verified fixed.

@nikhilmandlik
Copy link
Contributor

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.

7 participants