-
Notifications
You must be signed in to change notification settings - Fork 11.7k
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
Transformations: Move debug to drawer #76281
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.
Works great overall! The debug tooltip seems to not hide after closing the drawer though, and there is an unused variable now.
Screen.Recording.2023-10-11.at.09.55.07.mov
@@ -122,7 +122,6 @@ export const TransformationOperationRow = ({ | |||
)} | |||
<QueryOperationToggleAction | |||
title="Debug" | |||
disabled={!isOpen} |
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.
I think with the removal of this we can also remove the arg in renderActions
since isOpen
is now unused.
Nice catch and what and odd issue! I'll see what is going on with the tooltip. |
@mdvictor I've been looking into it and I think that the issue with the debug tooltip is broader between tooltip/drawer interaction. I think that's enough to receive it's own PR honestly. WDYT? |
Actually saw an issue similar to this where putting a fix broke datalinks 😱. I think we very much want to avoid that outcome and make sure we address the tooltip issue overall 😄 |
Sounds like a good plan, I agree! |
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.
Was using the debug extensively yesterday, and this would definitely have helped!
* Move debug to drawer * Prettier * Remove render actions arg * Remove unused import
* Move debug to drawer * Prettier * Remove render actions arg * Remove unused import
What is this feature?
This PR moves the transformation debug interface into a Drawer component for better usability.
Why do we need this feature?
Current transformation debug interface is inline and takes up a large amount of space within an already space constrained interface.
Which issue(s) does this PR fix?:
Fixes #74206
Special notes for your reviewer:
Please check that: