Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 355413 - Auto activation delay problems in EGL>Preferences>Content Assist
Summary: Auto activation delay problems in EGL>Preferences>Content Assist
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Xiao Bin Chen CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-22 12:38 EDT by broy2 CLA
Modified: 2017-02-23 14:14 EST (History)
1 user (show)

See Also:

Attachments
Before (48.29 KB, image/png)
2011-08-22 12:40 EDT, broy2 CLA 		no flags Details
After (48.23 KB, image/png)
2011-08-22 12:41 EDT, broy2 CLA 		no flags Details
Patch for ca preference page. (1.42 KB, patch)
2011-08-22 22:27 EDT, Xiao Bin Chen CLA 		lasher: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description broy2 CLA 2011-08-22 12:38:58 EDT 
Build Identifier: 20110819

The Auto activation delay field has a couple of problems.
1.  The default is 500, so the field has room for 3 digits. But if I change the delay to 3 and OK to exit, the next time I open the preferences, the field has room for only 3 digit. It looks like it will accept more than 1 digit, but only 1 is displayed.
2.  When the field is blanked out, OK is disabled and the message displays:  positive integer must be specified.  But if I type in a 0, OK is enabled and the message goes away.  0 should behave like a blank.

Reproducible: Always

Steps to Reproduce:
1.EGL>Preferences>Content Assist
2.Change Auto activation delay to 3 and OK to exit.
3.Open preferences again.
4.The Auto activation delay field displays only 1 digit. 
5.Blank out the field and get error. Enter a 0 and the error goes away.
Comment 1 broy2 CLA 2011-08-22 12:40:54 EDT 
Created attachment 201937 [details]
Before
Comment 2 broy2 CLA 2011-08-22 12:41:15 EDT 
Created attachment 201938 [details]
After
Comment 3 broy2 CLA 2011-08-22 12:41:39 EDT 
Screen shots of before and after attached.
Comment 4 Xiao Bin Chen CLA 2011-08-22 22:25:30 EDT 
Hi tony.
The patch was provided. Please take a look and apply it.
Comment 5 Xiao Bin Chen CLA 2011-08-22 22:27:34 EDT 
Created attachment 201964 [details]
Patch for ca preference page.

The patch 
1>disabled the auto-size of the textbox.
2>consider 0 condition.
Comment 6 Tony Chen CLA 2011-08-22 22:35:01 EDT 
code committed
Comment 7 broy2 CLA 2011-08-23 16:56:58 EDT 
20110823
Comment 8 broy2 CLA 2011-08-23 16:57:12 EDT 
Verified