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.

Bug 32637 - [TTV] TreeTable does not work in 1.4.2 GTK+ L&F
Summary: [TTV] TreeTable does not work in 1.4.2 GTK+ L&F
Status: VERIFIED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Explorer (show other bugs)
Version: 3.x
Hardware: All All
: P3 blocker (vote)
Assignee: Jiri Rechtacek
URL:
Keywords: JDK_SPECIFIC, L&F
: 33381 (view as bug list)
Depends on:
Blocks:
 
Reported: 2003-04-04 15:43 UTC by Jesse Glick
Modified: 2008-12-22 16:42 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Stack trace (1.92 KB, text/plain)
2003-04-04 15:43 UTC, Jesse Glick
Details
a workaround Gtk bug 4844870 (4.38 KB, patch)
2003-04-10 14:56 UTC, Jiri Rechtacek
Details | Diff
jar to lib/patches (101.92 KB, application/octet-stream)
2003-04-10 14:59 UTC, Jiri Rechtacek
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2003-04-04 15:43:13 UTC
[dev 030402] Just throws an exception. E.g. try to
open Tools -> Options.
Comment 1 Jesse Glick 2003-04-04 15:43:47 UTC
Created attachment 9711 [details]
Stack trace
Comment 2 Jesse Glick 2003-04-04 15:44:15 UTC
Also check in 3.5, probably the same.
Comment 3 Jiri Rechtacek 2003-04-04 16:33:27 UTC
Defect: UIManager.getColor("Table.selectionBackground") returns null.
The fix would be easy. Jesse, is it so urgent to integrate to nb3.5?
Comment 4 _ tboudreau 2003-04-08 12:57:28 UTC
I just filed JDK bug 4844816 for another problem in GTK L&F.  They
will need to fix this - it's an incompatibility between JTable and 
the GTK L&F - either JTable should test to see if UIManager.getColor()
is returning null for this arg, or (better), the L&F should use 
UIManager.  I'd say file a JDK bug.  
Comment 5 Jiri Rechtacek 2003-04-08 15:35:28 UTC
I filed a bugreport 4844870 as a blocker this issue.
Comment 6 _ tboudreau 2003-04-10 11:02:26 UTC
Jirko, if you need a temporary workaround for this issue, add it
after the code in Main that calls NbTheme, smth like:

Object o = UIManager.get ("Table.whatever");
if (o == null) UIManager.put (new ColorUIResource (whatever));

That will at least get it working so GTK L&F is usable on 3.5 (yes,
they need to fix it for the release).
Comment 7 Jiri Rechtacek 2003-04-10 14:56:18 UTC
Created attachment 9846 [details]
a workaround Gtk bug 4844870
Comment 8 Jiri Rechtacek 2003-04-10 14:59:19 UTC
Created attachment 9847 [details]
jar to lib/patches
Comment 9 Jesse Glick 2003-04-10 15:03:28 UTC
Not sure why this was marked as being a cross-platform bug, unless a
similar problems happens on other operating systems...?
Comment 10 Jiri Rechtacek 2003-04-10 15:07:40 UTC
The attached workaround suppresses the thrown exception when
Tool|Options dialog is being opened. It doesn't fix the reported jdk's
 bug 4844870 but only its known symptom in TreeTableView. I'm not sure
if it's enough to integrate to nb code line
Comment 11 Marian Mirilovic 2003-04-10 15:43:28 UTC
reproducible on Sol8/CDE
Comment 12 Jan Chalupa 2003-04-10 20:13:21 UTC
A comment from the JDK engineer working on the fix of #4844870:

----------------------%<-------------------------
I can fix the setSelectionForeground/setSelectionBackground color, but
then you guys are invoking setGridColor(null) from TreeTableView which
will throw an  IllegalArgumentException:

java.lang.IllegalArgumentException: New color is null
	at javax.swing.JTable.setGridColor(Unknown Source)
	at
org.openide.explorer.view.TreeTableView.initializeTreeTable(TreeTableView.java:292)

Which means you guys will need to change your code.
Is that acceptable?
----------------------%<-------------------------

Jirka, could you please re-evaluate?
Comment 13 Jan Chalupa 2003-04-15 11:09:23 UTC
A fix for #4844870 should be provided in final version of 1.4.2.
Nevertheless, the "workaround" proposed by Jiri should be integrated
into trunk, because the current code is depending on an assumption
which is not guaranteed by Swing.

No longer urgent for 3.5. P2 -> P3.
Comment 14 Jiri Rechtacek 2003-04-15 15:11:59 UTC
Proposed checks were added to all calling UIManger for colors.
Integrated to maintrunk.

Date: 03/04/15 07:03:53
Revision  Path
1.23      openide/src/org/openide/explorer/view/NodeRenderer.java
1.21      openide/src/org/openide/explorer/view/TableSheet.java
1.10      openide/src/org/openide/explorer/view/TableSheetCell.java
1.25      openide/src/org/openide/explorer/view/TreeTable.java
1.46      openide/src/org/openide/explorer/view/TreeTableView.java
Comment 15 Marian Mirilovic 2003-05-05 08:35:47 UTC
*** Issue 33381 has been marked as a duplicate of this issue. ***
Comment 16 Jan Chalupa 2003-05-05 08:42:26 UTC
*** Issue 33381 has been marked as a duplicate of this issue. ***
Comment 17 Marian Mirilovic 2005-07-13 13:24:06 UTC
closed