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

actionCollection update event looping when open View Large in Display layout #64 #3959

Closed
nikhilmandlik opened this issue Jun 21, 2021 · 2 comments · Fixed by #3989
Closed

Comments

@nikhilmandlik
Copy link
Contributor

nikhilmandlik commented Jun 21, 2021

Description:

When click on 'View Large' in Display layout updateActionItems event triggers in loop and only stop if we navigate away from Display layout

Testing instructions: (local only)

  1. add console.log('ObjectFrame, updateActionItems'); in openmct/src/ui/components/ObjectFrame.vue, inside function updateActionItems(actionItems) {
  2. create/use Display layout
  3. add new /use existing object which has 'View Large' Screen Shot 2021-06-21 at 10 53 56 AM
  4. click on 'View Large' and observer log updateActionItems in console
  5. close large view still log updateActionItems in console keeps printing.
  6. navigate away from Display layout
  7. now logging stops.

Expected Result:

should not loop update event

Actual Result:

looping update event

Console Errors: N/A

Screenshots:

Screen Shot 2021-06-21 at 11 04 51 AM

@nikhilmandlik
Copy link
Contributor Author

update: looks like function updateActionItems inside preview-header.vue hides this.actionCollection.hide(HIDDEN_ACTIONS); some actions and actionCollection emits event update which again triggers this updateActionItems function again and so on.

@akhenry
Copy link
Contributor

akhenry commented Jul 12, 2021

Verified Fixed.

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

Successfully merging a pull request may close this issue.

3 participants