Bug 323941 - [organize imports] "Add Import" dialog should have hint about type filters
Summary: [organize imports] "Add Import" dialog should have hint about type filters
Status: ASSIGNED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: JDT-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 48736
Blocks:
  Show dependency tree
 
Reported: 2010-08-30 05:29 EDT by Christian CLA
Modified: 2020-05-04 03:06 EDT (History)
3 users (show)

See Also:

Attachments
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 Christian CLA 2010-08-30 05:29:14 EDT 
Build Identifier: I20100608-0911

"Add import" considers classes from com.sun packages as every other class.
Though very seldom one is interested in using non api classes.

It would improve speed to not show a dialog for choosing between com.sun.* classes if only a single normal (not com.sun.*) class exists. 
The classes of com.sun are normally never used, so there is no need to consider them for "Add Import" function.

Reproducible: Always

Steps to Reproduce:
1. tpye "File" in the source file
2. hit ctrl+Shift+M
3. A window pops up giving you a choice ... that you don't want...
Comment 1 Markus Keller CLA 2010-08-30 12:34:12 EDT 
You can add a Type Filter for com.sun.* classes in the preferences (Java > Appearance > Type Filters).

Keeping this bug for adding some affordance for this in the dialog (e.g. description with a link to the preference page). See also bug 48736, which could influence this.
Comment 2 Christian CLA 2010-08-31 11:53:32 EDT 
It might be a good choice for a default setting then.

I don't know how eclipse policy is for default settings, but this seems like an advantageous setting for most users.
Comment 3 Noopur Gupta CLA 2020-05-04 03:05:12 EDT 
*** Bug 562752 has been marked as a duplicate of this bug. ***