This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | SearchBar and ReplaceBar have a different close button UI to nb's typical close button | ||
---|---|---|---|
Product: | editor | Reporter: | Milutin Kristofic <mkristofic> |
Component: | Search | Assignee: | Milutin Kristofic <mkristofic> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | apireviews, jglick, mkristofic, saubrecht |
Priority: | P3 | Keywords: | API_REVIEW_FAST |
Version: | 7.1 | ||
Hardware: | PC | ||
OS: | Linux | ||
Issue Type: | ENHANCEMENT | Exception Reporter: | |
Attachments: |
Close Button API PATCH
Close Button UI Close Button API PATCH 2 Close Button API PATCH Close Button API PATCH |
Description
Milutin Kristofic
2011-09-05 14:22:42 UTC
Created attachment 110403 [details]
Close Button API PATCH
I am proposing an attached patch. It should unify close buttons in Netbeans
I created simple CloseButtonFactory with two public static methods createCloseButton and createBigCloseButton. They simple create JButton with specific icons for each look and feel.
I just moved close button implementation from CloseButtonTabbedPane to new CloseButtonFactory.
Created attachment 110404 [details]
Close Button UI
I also attached a screenshot of new Close Button UI in metal look and feel
By the way do not use the IDE's "Export Diff" when creating patches for review. Ensure that diff.git=1 is configured in ~/.hgrc and use 'hg diff'. See: http://wiki.netbeans.org/HgHowTos#Develop_API_review_patches_using_MQ Also set <date> in apichanges.xml to the date you actually expect to integrate, typically one week from the day you initiate a review. <version> and @since should be accompanied by an actual increment in the version in manifest.mf (or project.properties if using spec.version.base), and a matching use of the new version in project.xml of the client module (here editor). The API change entry should also use <class> so as to link to the new Javadoc. Is there any special button for GTK L&F? OK if not, just curious. Created attachment 110485 [details]
Close Button API PATCH 2
Thank you for notes, Jesse. I attached a new hg diff as a patch, changed date to 14 Sept, changed manifest.mf to 7.38, changed api version in editor, added class to apichange.
I also find icons for GTK in Tab Control, so I added them to close button api. They were not in org.openide.awt.
Note that you are still not using Git-format diffs. Does not matter in this case unless someone wanted to actually try your patch, in which case the binary file adds would not be available, but generally good practice to set this option. Also the revised patch is missing the new class. Created attachment 110515 [details]
Close Button API PATCH
Thank you, Mea Culpa.
I attached Close Button API Patch - it's a git diff with all files.
[JG01] You may be able to avoid copying all those icons - o.n.swing.tabcontrol has a dependency on openide.awt, so they could just be moved into openide.awt and o.n.swing.tabcontrol could refer to the new resource paths. Created attachment 110569 [details]
Close Button API PATCH
Alright, I deleted same icons from tabcontrol module and changed path for icons in tabcontrol module. I also increased dependecy on org.openide.awt
I pushed the change in http://hg.netbeans.org/jet-main/rev/e591254a6161 Integrated into 'main-golden' Changeset: http://hg.netbeans.org/main-golden/rev/e591254a6161 User: Milutin Kristofic <mkristofic@netbeans.org> Log: #201639 - SearchBar and ReplaceBar have a different close button UI to nb's typical close button |