Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 350733 - Breakpoint condition editor needs a height hint
Summary: Breakpoint condition editor needs a height hint
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 trivial (vote)
Target Milestone: 3.8 M1   Edit
Assignee: JDT-Debug-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-29 13:53 EDT by Michael Rennie CLA
Modified: 2011-07-06 03:29 EDT (History)
1 user (show)

See Also:


Attachments
screenshot (70.72 KB, image/png)
2011-06-29 13:53 EDT, Michael Rennie CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2011-06-29 13:53:26 EDT
Created attachment 198849 [details]
screenshot

code from HEAD

while testing switch-on-String support in conditional breakpoints I found that if you have a really long condition and open the breakpoint properties dialog, it will be the entire height of the screen before it will scroll. Consider the following condition:

String test = "two";
switch (test) {
case "one":
	System.out.println("one from condition");
    return true;
case "two":
	System.out.println("two from condition");
	String innertest = "innerone";
	switch(innertest) {
	case "innerone": {
		System.out.println("innerone from inner condition");
		return true;
	}
	default: {
		System.out.println("fail from inner condition");
		return false;
	}
	}
    return true;
case "three":
	System.out.println("three from condition");
    return true;
case "four":
	System.out.println("four from condition");
    return true;
case "five":
	System.out.println("five from condition");
    return true;
default:
	System.out.println("Fail from condition");
	return false;
}

After it opens if you try to resize the properties dialog the condition text editor does not resize, instead the dialog scrolls, which is annoying.
Comment 1 Michael Rennie CLA 2011-07-04 10:57:11 EDT
There already was height hint code there, just commented out.

Fixed in HEAD