TreeGrid SelectionChanged

yavery · #1 29th Mar 2012, 08:59

Hi,

We've finally confirmed an issue with the TreeGrid SelectionChanged event. The problem can be seen
on either IE8 or IE9. It's not reproducible every single time, but if you play around with selections,
you'll see it ... Actually, it seems to be even more apparent under IE9.

We are seeing this using SmartClient_v82p_2012-03-26 (LGPL).

The problem is that the "clicking" of selection checkboxes within a TreeGrid sometimes reverts to
the value it had prior to the click (as if 2 clicks were digested). You can see the checkbox blink.

We used the following test bed to reproduce ... If you click, click, click, click again in different TreeGrid
checkboxes, you'll notice the issue ...

Code:

<HTML>
	<HEAD>
		<META HTTP-EQUIV="X-UA-Compatible" CONTENT="IE=9" />
		<LINK REL="shortcut icon" HREF="images/logo/favicon.ico" type="image/x-icon"/>
		<LINK REL="icon" HREF="images/logo/favicon.ico" type="image/x-icon"/>
<SCRIPT>
window.isomorphicDir = 'isomorphic/';
</SCRIPT>

		<SCRIPT SRC="isomorphic/system/modules/ISC_Core.js"></SCRIPT>
		<SCRIPT SRC="isomorphic/system/modules/ISC_Foundation.js"></SCRIPT>
		<SCRIPT SRC="isomorphic/system/modules/ISC_Containers.js"></SCRIPT>
		<SCRIPT SRC="isomorphic/system/modules/ISC_Grids.js"></SCRIPT>
		<SCRIPT SRC="isomorphic/system/modules/ISC_Forms.js"></SCRIPT>
		<SCRIPT SRC="isomorphic/system/modules/ISC_DataBinding.js"></SCRIPT>
		<SCRIPT SRC="isomorphic/skins/MEI/load_skin.js"></SCRIPT>

		<TITLE>Investigation - selection changed</TITLE>
	</HEAD>

	<BODY STYLE="overflow:hidden">
		<SCRIPT>
		var employeeTree = isc.TreeGrid.create({
			selectionAppearance:"checkbox",
		    data: isc.Tree.create({
		        modelType: "parent",
		        nameProperty: "Name",
		        idField: "EmployeeId",
		        parentIdField: "ReportsTo",
		        data: [
		            {EmployeeId:"4", ReportsTo:"1", Name:"Charles Madigen"},
		            {EmployeeId:"188", ReportsTo:"4", Name:"Rogine Leger"},
		            {EmployeeId:"189", ReportsTo:"4", Name:"Gene Porter"},
		            {EmployeeId:"265", ReportsTo:"189", Name:"Olivier Doucet"},
		            {EmployeeId:"264", ReportsTo:"189", Name:"Cheryl Pearson"}
		        ]
		    }),
		    width: 500,
		    height: 400
		});

		employeeTree.getData().openAll();

		</SCRIPT>
	</BODY>
</HTML>

Thanks for looking into this.

Kind regards,

Isomorphic · #2 30th Mar 2012, 01:38

Thanks for the report - this had already been fixed in 8.3d and the fix has now been backported to 8.2p. Please retest with a nightly build after tomorrow.

yavery · #3 2nd Apr 2012, 06:40

Hi,

Just downloaded and installed latest 8.2p (2012-04-02) and I still have this issue. Clearly visible on IE9.

Can someone please look into this ...

Thanks,

Isomorphic · #4 2nd Apr 2012, 11:12

Hm - there's a problem with the META tag here - if you run your sample code without that tag, whether in native IE or not, it works fine - no apparent double-clicks, no flashing - we're looking further into it and will update this thread when we have something to add.

yavery · #5 2nd Apr 2012, 12:01

Just noticed it. But when you put it, it fails ... Strange indeed ...

While you're looking on your side, I'm also looking on my side as to WHY I added that
line to our code base and see if I really need it.

Thanks,

yavery · #6 3rd Apr 2012, 04:18

Hi,

I just finished investigating on my side, and I have some CSS/HTML (non Isomorphic related)
that uses IE9 features which require true IE9, not IE7 or IE8 compatibility mode.

Therefore, I must leave that META tag in there, to make sure users are not trying to run my app
in IE7 or IE8 compatibility mode.

Does Isomorphic support IE9 other than in IE7/IE8 compatibility mode ?

Thanks,

Isomorphic · #7 3rd Apr 2012, 10:17

We support using the HTML5 Doctype in IE9 as of 8.2, yes. Using this should be equivalent to your meta tag *plus* enable HTML5 features, but we're looking into whether there's some kind of subtle difference.

yavery · #8 4th Apr 2012, 06:22

I removed the <META> tag but that doesn't prevent a user from manually setting
his IE9 browser to which ever compatibility mode he wants, and this is what's breaking ...

The only mode which doesn't break is "Quirks Mode".

Please provide a workaround for this issue that works in all cases.

Thanks,

Isomorphic · #9 4th Apr 2012, 08:26

It turns out, there is actually a SmartClient bug here - or at least, an unexpected behavior we can replicate fairly easily with TreeGrid in IE9 with your META tag (or with the html5 doctype) - we're taking a look and will update this thread when there's something to update it with.

yavery · #10 4th Apr 2012, 08:29

Great. Thanks for the heads up. I'll be waiting for this one. Thanks!

Thread Tools	Search this Thread
Show Printable Version Email this Page	Search this Thread: Advanced Search

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Tree TreeGrid and hierarchical data problem	siegersallee	Smart GWT Technical Q&A	11	5th Feb 2013 07:28
TreeGrid selectionChanged	acarur01	Technical Q&A	2	22nd Jul 2011 01:22
TreeGrid and Tree	jmlordo	Smart GWT Technical Q&A	5	15th Jan 2010 09:52
TreeGrid with DataSource initialized via setTestData	pedjak	Smart GWT Technical Q&A	1	30th Jun 2009 16:49
Proble TreeGrid with Firefox 3.0.5	mvarlic	Smart GWT Technical Q&A	2	3rd Feb 2009 13:40