Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 262155 - [CellEditor] Keyboard Navigation Up/Down change the table selection into the wrong cell
Summary: [CellEditor] Keyboard Navigation Up/Down change the table selection into the ...
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: JFace (show other bugs)
Version: 1.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.2 M5   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 261466 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-01-23 07:38 EST by Wen Ai CLA
Modified: 2009-02-05 12:49 EST (History)
1 user (show)

See Also:

Attachments
Snippet to reproduce (13.84 KB, text/plain)
2009-01-23 07:38 EST, Wen Ai CLA 		no flags Details
Screenshot with TextCellEditor (7.66 KB, image/png)
2009-01-23 07:53 EST, Wen Ai CLA 		no flags Details
Screenshot with ComboBoxCellEditor (8.34 KB, image/png)
2009-01-23 07:54 EST, Wen Ai CLA 		no flags Details
Screenshot with the following uncorrect effect (8.04 KB, image/png)
2009-02-05 09:12 EST, Wen Ai CLA 		no flags Details
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 Wen Ai CLA 2009-01-23 07:38:23 EST 
Created attachment 123505 [details]
Snippet to reproduce
Comment 1 Wen Ai CLA 2009-01-23 07:53:55 EST 
Created attachment 123507 [details]
Screenshot with TextCellEditor
Comment 2 Wen Ai CLA 2009-01-23 07:54:18 EST 
Created attachment 123508 [details]
Screenshot with ComboBoxCellEditor
Comment 3 Wen Ai CLA 2009-01-23 07:55:58 EST 
Steps to reproduce:

1) Open the original rap-application "org.eclipse.rap.demo"
2) Take this snippet TableViewerTab.java under the package org.eclipse.rap.demo.controls instead of the old one
3) Activate a TextCellEditor(column lastname, column age), edit some text
4) Navigate with the key Up/Down

The table selection changes its position to another cell, but the focus stays always where the cell editor was activated.

One similar bug with the number 261466 has reported the same situation, but with ComboBoxCellEditor. In ComboBoxCellEditor it happens by me too.

I have tried this also with the CheckBoxCellEditor, under CheckBoxCellEditor has no such problem. 

 

Reproduce under FF 3.0.5, Rap 1.2M4 without all rap-jars, and all rap projects from CVS HEAD.
Comment 4 Ralf Sternberg CLA 2009-01-30 09:24:20 EST 
*** Bug 261466 has been marked as a duplicate of this bug. ***
Comment 5 Ralf Sternberg CLA 2009-02-05 07:38:03 EST 
The problem was that the client side Table implementation did not sort out bubbling events from nested widgets. Client-side event bubbling is crucial for things like focus control, default button, closing dialogs on Esc etc. to work.
Fixed in CVS.
Comment 6 Wen Ai CLA 2009-02-05 09:12:26 EST 
Created attachment 124800 [details]
Screenshot with the following uncorrect effect
Comment 7 Wen Ai CLA 2009-02-05 09:23:15 EST 
There comes an another unexcepted effect with this snippet after the bugfix. The cell with ComboBoxCellEditor (in the first column) would not be disselected after leaving this cell. See the last screenshot. 

To reproduce:
1. the same snippet
2. current version from CVS at 15:00 (german time)
Comment 8 Ralf Sternberg CLA 2009-02-05 12:49:13 EST 
This was a regression caused by the fix for bug 261647. Fixed in CVS.