ComboBoxCell exception on IE8

[mrjoe]

Hello

I added ComboBox component to the PagingToolBar component to enable user to change number of displayed items in the table. Everytime user changes table row count, he gets following error in IE8 / IE7.

 number: -2146826178 description:
Can't move focus to the control because it is invisible, not enabled, or of a type that does not accept the focus.

I found the problem in ComboxBoxCell.java

protected void onViewClick(final XElement parent, NativeEvent event, boolean focus, boolean takeSelected) {    

    // .... removed some lines for better readability ....

    if (focus) {
      Scheduler.get().scheduleDeferred(new ScheduledCommand() {


        @Override
        public void execute() {
          if (GXTLogConfiguration.loggingIsEnabled()) {
            logger.finest("onViewClick parent.focus()");
          }

          getInputElement(parent).focus(); // <----- THIS WILL CRASH !!!
        }
      });
    }


  }

I only crashes in IE that cannot handle focusing of the disabled items correctly. This bug does not show in Opera, Firefox or Chrome (Chromium). But because the toolbar is disabled while waiting for loader to finish loading data, focusing of the combox will throw exception.

[Colin Alworth]

Thanks for the bug report - we see this crop up in many ways, but unfortunately our attempts to rework this at a low level have been stymied by the fact that Element.focus must be final, so we can't add an extra isIE and isVisible check there.

Can you give a little more detail on your use case? The ComboBox has been added to the PagingToolbar, but is invisible/disabled/detached? How is it the user can see it then, or can interact with it?

[mrjoe]

ComboBox is visible and enabled. I do not disable it from my code. It is only disabled by the parent toolbar while waiting for loader to finish loading data.

[kpiskunov]

I'm seeing the same exception in IE 7 when navigating around the editable tree grid:
http://www.sencha.com/examples/#Exam...itabletreegrid

It happens when I click on Author/Genre fields repeatedly. In our code we use a TreeGrid with a ComboBox and the exception pops-up as well.

[Colin Alworth]

We've just made a few changes that should deal with this and related IE focus issues. Can you still reproduce the issue on our nightly build at http://staging.sencha.com:8080/examples-dev/ ?

[mrjoe]

Thank you for your effort! I will test it in my project and will report back. Is it sufficient to update GXT to SVN revision 2582?

[kpiskunov]

I'm not encountering the issue in the nightly build.

[Colin Alworth]

Great - I'm marking this issue as FIXED, please feel free to post again if you encounter this again.