Forum

November 2nd, 2014
A A A
Avatar

Lost password?
Advanced Search

— Forum Scope —




— Match —





— Forum Options —





Minimum search word length is 3 characters - maximum search word length is 84 characters

The forums are currently locked and only available for read only access
sp_Feed Topic RSS sp_TopicIcon
MSIE 8 latest Edition 8.0.6001.18813, context menu fails
14/08/2009
02:47
Avatar
hpoe
hpoe
Member
Members
Forum Posts: 29
Member Since:
07/03/2009
sp_UserOfflineSmall Offline

After the automatic update of MSIE 8

in Demo "TreeGrid real world Example" the context menu is not functioning any more

15/08/2009
01:20
Avatar
tony
Sofia, Bulgaria
Moderator
Members

Moderators
Forum Posts: 7721
Member Since:
30/10/2007
sp_UserOfflineSmall Offline

Hello,

Thnaks. I confirm this, but I think:

1. Some things in IE8 are changed.

2. It is quite possible that the problem is in qontent menu plugin which is used for this purpose.

In all cases I will investigate this.

Bets Regards

Tony

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

15/08/2009
03:48
Avatar
hpoe
hpoe
Member
Members
Forum Posts: 29
Member Since:
07/03/2009
sp_UserOfflineSmall Offline

Hello Tony

thanks, for replying.

onContextMenu: function(e, menu) {
  td = e.target || e.srcElement;
  ptr = $(td).parents("tr.jqgrow")[0];
  ci = !$(td).is('td') ? $(td).parents("td:first")[0].cellIndex : td.cellIndex;
  if($.browser.msie) {
    ci = $.getAbsoluteIndex(ptr,ci);
  }

  if( ci >= 1 ) { .........

Since MSIE 8 did not like $.getAbsoluteIndex(ptr,ci), I first tested for the browser version, and it worked in MSIE 8 but not in 7 and 6. So I tried to just leave out the MSIE specific code and it worked again.

It seems that MS has changed something significant in that area. As I am not a jQuery expert I could by wrong, but I simply tested the code step by step and so I tried to find out where it breaks.

Hope this helps and that it is not a minor version problem, that arrived with the latest MS patches.

Regards

hpoe

15/08/2009
03:48
Avatar
hpoe
hpoe
Member
Members
Forum Posts: 29
Member Since:
07/03/2009
sp_UserOfflineSmall Offline

Hello Tony

thanks, for replying.

onContextMenu: function(e, menu) {
  td = e.target || e.srcElement;
  ptr = $(td).parents("tr.jqgrow")[0];
  ci = !$(td).is('td') ? $(td).parents("td:first")[0].cellIndex : td.cellIndex;
  if($.browser.msie) {
    ci = $.getAbsoluteIndex(ptr,ci);
  }

  if( ci >= 1 ) { .........

Since MSIE 8 did not like $.getAbsoluteIndex(ptr,ci), I first tested for the browser version, and it worked in MSIE 8 but not in 7 and 6. So I tried to just leave out the MSIE specific code and it worked again.

It seems that MS has changed something significant in that area. As I am not a jQuery expert I could by wrong, but I simply tested the code step by step and so I tried to find out where it breaks.

Hope this helps and that it is not a minor version problem, that arrived with the latest MS patches.

Regards

hpoe

15/08/2009
05:13
Avatar
tony
Sofia, Bulgaria
Moderator
Members

Moderators
Forum Posts: 7721
Member Since:
30/10/2007
sp_UserOfflineSmall Offline

Hello,

Thanks. Actually the problem is into the code of the demo file. I forgot to update the $.getAbsoluteIndex to $.jgrid.getAbsoluteIndex (since of namespace conflict with other plugins). Now evrething work fine in all IE  Wink - you can test this.

Thank you very much for this again.

Best Regards

Tony

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

16/08/2009
06:05
Avatar
hpoe
hpoe
Member
Members
Forum Posts: 29
Member Since:
07/03/2009
sp_UserOfflineSmall Offline

Hello Tony

yes, it is working in the demo now. Please don't forget to correct the example code as well.

In my own project it is not working with your solution, but I used a different way to attach the content menu to the grid, I followed the approach which was published in the integration section in spanish, where the context menu is attached in the afterInserRow event.

If I leave out the MSIE specific code it works in my project.

Thanks for your work.

Regards

hpoe

17/08/2009
06:55
Avatar
tony
Sofia, Bulgaria
Moderator
Members

Moderators
Forum Posts: 7721
Member Since:
30/10/2007
sp_UserOfflineSmall Offline

Hello,

Demo upadted.

Regards

Tony

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
93 Guest(s)

Currently Browsing this Page:
1 Guest(s)

Top Posters:

OlegK: 1255

markw65: 179

kobruleht: 144

phicarre: 132

YamilBracho: 124

Renso: 118

Member Stats:

Guest Posters: 447

Members: 11373

Moderators: 2

Admins: 1

Forum Stats:

Groups: 1

Forums: 8

Topics: 10592

Posts: 31289

Newest Members:

, razia, Prankie, psky, praveen neelam, greg.valainis@pa-tech.com

Moderators: tony: 7721, Rumen[Trirand]: 81

Administrators: admin: 66

Comments are closed.
Privacy Policy   Terms and Conditions   Contact Information