-
Notifications
You must be signed in to change notification settings - Fork 0
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
VUU-33 add context for layouts #35
VUU-33 add context for layouts #35
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
@@ -81,7 +74,7 @@ export const ShellWithNewTheme = () => { | |||
} | |||
return menuDescriptors; | |||
}, | |||
async (action: MenuActionClosePopup) => { | |||
(action: MenuActionClosePopup) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a MenuActionClose action has an options object. I made a change to add an attribute to that options object controlledComponetId
is the id of the component controlled by the tab from which the save layout menu was popped up. That gives you the id of the element we need to capture teh screen grab from. Will avoid that slightly fragile
getElementsByClassName("vuuShell-content")[0] to locate the element.
It can be passed as a prop straight into the SaveLayoutPanel
…ttps://github.com/ScottLogic/finos-vuu into feature/VUU-33-Layout-Manager-Context
f7ceec0
into
feature/layout-management/layout-management-feature
No description provided.