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

[LAD Tables] Configuration not changed/persisted when toggling "Autosize" for columns #6636

Closed
2 of 7 tasks
unlikelyzero opened this issue May 2, 2023 · 1 comment · Fixed by #6637
Closed
2 of 7 tasks
Labels
Milestone

Comments

@unlikelyzero
Copy link
Contributor

unlikelyzero commented May 2, 2023

Summary

There are some cases where you can't autosize the columns. This is sometimes avoided if there are hidden columns or if you were to hide/show columns and then autosize.

Expected vs Current Behavior

When clicking on the 'Autosize' LAD Table button, we should expect it to do something. Either equal width columns or autosized columns.

Steps to Reproduce

  1. View LAD Table and copy it's id from the url
  2. in the console, await openmct.objects.get('[thatIdThatWeJustTalkedAbout]')
  3. right click the JSON and store as a global variable
  4. openmct.objects.mutate(temp1, 'configuration', {})
  5. refresh the view, try to autosize the columns
  6. make sure you can still hide/show columns as expected

I'm not sure how we got into this state before, but this is a solid way to see if a fix for it works.

Environment

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

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? if you show or hide columns then autosize, it works
  • 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

@jvigliotta jvigliotta changed the title LAD Tables do not expand under certain circumstances [LAD Tables] Configuration not changed/persisted when toggling "Autosize" for columns May 2, 2023
@ozyx ozyx closed this as completed in #6637 May 2, 2023
@unlikelyzero unlikelyzero added this to the Target:2.2.3 milestone May 8, 2023
@charlesh88
Copy link
Contributor

Testathon 2023-05-08 verified fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants