Bug 438554 - [Shell] traverseGroup should be implemented properly
Summary: [Shell] traverseGroup should be implemented properly
Status: ASSIGNED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 2.3   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 438634 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-06-30 09:25 EDT by Adrian Stefanescu CLA
Modified: 2020-09-02 03:25 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Stefanescu CLA 2014-06-30 09:25:20 EDT
Right now the traverseGroup from Shell has a "fake" implementation that doesn't work right

 private boolean traverseGroup( boolean next ) {
    // TODO [rh] fake implementation
    boolean result = false;
    if( getChildren().length > 0 ) {
      result = getChildren()[ 0 ].forceFocus();
    }
    return result;
  }

It just tries to put the focus on the first control it finds. Because of this, the result of opening dialogs is not the same in RAP and RCP. For example the ShowView dialog in RCP has the focus in the top Text control but the RAP ShowView doesn't because the focus control in the RAP is a Composite.
Comment 1 Ivan Furnadjiev CLA 2014-07-04 05:38:02 EDT
*** Bug 438634 has been marked as a duplicate of this bug. ***
Comment 2 Andreas Blochberger CLA 2020-09-02 03:25:44 EDT
When the whole method is removed, it works as expected. The call in setEnabled() can be replaced with setFocus()