Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 351849

Summary:

[Table] JavaScript Error in context of cell tooltips

Product:

[RT] RAP

Reporter:

Sabine Neubauer <neubauer>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

Version:

1.4

Target Milestone:

1.5 M1

Hardware:

All

OS:

All

Whiteboard:

sr141

Attachments:

Description	Flags
Used client.js	none
Proposed patch for v14_Maintenance branch	ivan: review?, rsternberg: review+

Description Sabine Neubauer

2011-07-12 10:14:30 EDT

Build Identifier: 

In our applications we have several tables with cell tooltips. Sometimes a javascript error occurs in the method Table.js#_onClientAreaMouseMove. Unfortunately, I am not able to reproduce the problem properly. Hence, I can currently not provide a small snippet.

The error message on IE is the following:
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0;
SLCC2; .NET
CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0;
InfoPath.3;
MS-RTC LM 8; .NET4.0C; .NET4.0E)
Timestamp: Wed, 6 Jul 2011 14:18:01 UTC

Message: 'null' is null or not an object
Line: 170
Char: 32390
Code: 0
URI: ...

Further information:
- RAP 1.4 RC2
- used browsers: InternetExplorer 8 and Firefox (3.6 or newer)

Reproducible: Sometimes

Comment 1 Sabine Neubauer

2011-07-12 10:18:31 EDT

Created attachment 199500 [details]
Used client.js

Comment 2 Ivan Furnadjiev

2011-07-13 02:11:10 EDT

Thanks for the new bug report, Sabine. Current information does not help us much to isolate the problem. Could you describe some steps that provoke the error in your application - is Table in popup/dialog/wizard, is this happen when disposing the table? Please run your application in Fifrefox and provide the complete error information from the error page.

Comment 3 Sabine Neubauer

2011-07-15 06:28:38 EDT

The only information I have at the moment:
The table is part of a view containing several sections which are always expanded.

Currently, I am not able to reproduce the bug. I don't think that's because it now works but I need further information from the end user who reported the problem to me. As soon I have further information I will let you know. Thanks in advance.

Comment 4 Sabine Neubauer

2011-07-22 03:01:24 EDT

I am now able to reproduce the problem very reliable.

1. Starts the RAP examples demo
2. Open your Internet Explorer and set the zoom factor to 110%
3. Now open the examples demo (it is important that the zoom factor is set before first access!)
4. Open the TableViewer "tab"
5. Move the mouse over the table in order to get cell tooltips
=> Java script error

Comment 5 Ivan Furnadjiev

2011-07-22 03:13:28 EDT

Thanks Sabine for the hint. I got the JavaScript error in IE9 with zoom level 110%.

Comment 6 Ivan Furnadjiev

2011-07-22 03:47:18 EDT

Created attachment 200152 [details]
Proposed patch for v14_Maintenance branch

This bug does not exist in CVS HEAD. The error appears when the last visible item is partly visible and you request a cell tooltip for it. The method Table.js#_getRowAtPoint returns null in this case. The patch checks if row is not null in Table.js#_onClientAreaMouseMove.

Comment 7 Sabine Neubauer

2011-07-22 04:56:31 EDT

(In reply to comment #6)
> Created attachment 200152 [details]
> Proposed patch for v14_Maintenance branch
> 
> This bug does not exist in CVS HEAD. The error appears when the last visible
> item is partly visible and you request a cell tooltip for it. The method
> Table.js#_getRowAtPoint returns null in this case. The patch checks if row is
> not null in Table.js#_onClientAreaMouseMove.

Thanks for the patch it solves the problem. But I think the reason for the error can also be another one. Because the problem occurs in our application also if the table has only one or two rows which are completely visible and the table has no scrollbar in this case. Precondition is to use a zoomed Internet Explorer.

Comment 8 Ivan Furnadjiev

2011-07-26 04:13:37 EDT

Applied patch to v14_Maintenance branch.