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

Fix breakpoint expression resizing #9527

Conversation

kenneth-marut-work
Copy link
Contributor

What it does

Fixes #9526 by adding a resize handler to the debug-breakpoint-widget's input editor. It also adds a resize emitter on the MonacoEditor so that resize events from a parent Widget can be forwarded when necessary.

How to test

Follow the steps outlined in #9526, and observe that the expression input field does resize when the the window is maximized

good-resizing

Review checklist

Reminder for reviewers

@vince-fugnitto vince-fugnitto added debug issues that related to debug functionality ui/ux issues related to user interface / user experience labels May 28, 2021
@kenneth-marut-work kenneth-marut-work force-pushed the breakpoint-input-resize branch from a1606d2 to e31d44e Compare June 1, 2021 16:52
@colin-grant-work colin-grant-work self-requested a review June 4, 2021 17:47
Copy link
Contributor

@colin-grant-work colin-grant-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirm that this change allows the breakpoint widget to resize with the editor widget.

Signed-off-by: Kenneth Marut <kenneth.marut@ericsson.com>
@kenneth-marut-work kenneth-marut-work force-pushed the breakpoint-input-resize branch from e31d44e to f206df3 Compare June 9, 2021 20:05
@kenneth-marut-work kenneth-marut-work merged commit 64e613f into eclipse-theia:master Jun 9, 2021
@vince-fugnitto vince-fugnitto added this to the 1.15.0 milestone Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debug issues that related to debug functionality ui/ux issues related to user interface / user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Breakpoint expression input field does not auto resize
3 participants