SelectionAppearance.CHECKBOX causing error in ListGrid

[GoogleCodeExporter]

Before your post : If you are not 100% sure that you've found an issue,
please post on the SmartGWT forums first. Upon confirmation you'll be
advised to create an issue.

What steps will reproduce the problem?
1. Create ListGrid
2. setFields
3. Set SelectionAppearance.CHECKBOX
4. move the mouse on the broken checkbox header column

What is the expected output? What do you see instead?
Expected is that the Table Header is showing a regular checkbox with grey 
background, moving the mouse over should not cause an error.
However it produces an error popup.

What version of the product are you using? On what operating system?
smartgwt-3.0.jar on Windows

What browser(s) does this happen in?  Are there any browsers where the
issue does not occur?
Occurs in all browsers I testet: both IE 8 or FF 10.0.4

Please provide any additional information below.
Screnshots attached.

Smallest piece of Code to reproduce in onModuleLoad():

    ListGrid list = new ListGrid();
    ListGridField field = new ListGridField("Operation");
    list.setFields(field);
    list.setSelectionAppearance(SelectionAppearance.CHECKBOX);
    list.setSelectionType(SelectionStyle.SIMPLE);
    list.show();

Original issue reported on code.google.com by joerg.ho...@googlemail.com on 26 Apr 2012 at 4:44

Attachments:

• SmartGWT-ListGrid-CheckboxSelection-Bug-Popup.png
• SmartGWT-ListGrid-CheckboxSelection-Bug.png

[GoogleCodeExporter]

This code does not reproduce a problem.  Most likely your actualy code involves 
invalid mixing of GWT and SmartGWT widgets.

If you disagree, add a test case that actually reproduces the problem against 
the latest patch build from smartclient.com/builds.

Marking Invalid until reproducible.

Original comment by smartgwt...@gmail.com on 27 Apr 2012 at 4:46

• Changed state: Invalid

[GoogleCodeExporter]

You are right. I created a new empty GWT Project and there is no error.
I checked what is causing the Problem and found that part of it is this line:

SC.setScreenReaderMode(true);

So if you add this the checkbox in the header disappears. This already appears 
as a bug to me.

However, this is still not causing the same problem I described. In my project 
we are also using gwt-platform so I suppose there is some problem as you 
suggested by "mixing GWT and SmartGWT".
However, this is why I used "list.show()" instead of adding it properly to the 
view.

I removed the SC.setScreenReaderMode(true); line from my app and now it works. 
However we have to check the impact on accessibility as we need to support 
screen readers.

Thanks a lot so far.

Original comment by joerg.ho...@googlemail.com on 27 Apr 2012 at 7:46

[GoogleCodeExporter]

public class Smartgwt_test implements EntryPoint {

  /**
   * This is the entry point method.
   */
  public void onModuleLoad() {

    SC.setScreenReaderMode(true);

    ListGrid list = new ListGrid();
    ListGridField field = new ListGridField("Operation");
    list.setFields(field);
    list.setSelectionAppearance(SelectionAppearance.CHECKBOX);
    list.setSelectionType(SelectionStyle.SIMPLE);
    list.show();
  }
}

Original comment by joerg.ho...@googlemail.com on 27 Apr 2012 at 7:47

[GoogleCodeExporter]

Just to clarify, this code still does not reproduce the problem *until you add 
gwt-platform*, right?

Original comment by smartgwt...@gmail.com on 28 Apr 2012 at 6:54

[GoogleCodeExporter]

A follow up: We were able to reproduce this issue and have made a change to 
resolve it. Should no longer be occurring in nightly builds cut from the 3.0p 
or 3.1d branch (as of May 17 or greater).

Changing status from invalid to fix.

Original comment by smartgwt...@gmail.com on 17 May 2012 at 12:53

• Changed state: Fixed

[GoogleCodeExporter]

Feedback:
We have upgraded to SmartGWT 3.1 and can confirm that this issue is resolved 
even with our entire technology stack (GWTP, OpenLayers, etc.).
Accessibility and keyboard control of Menü, etc. is now a lot better.

Thanks for your great support!

Original comment by joerg.ho...@googlemail.com on 29 Nov 2012 at 12:03