jQuery Grid Plugin - jqGridForum

Topic RSS

Related Topics sp_TopicIcon

two small bugs

Tags: focus tabindex

19/03/2014
13:21

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hello Tony,

I want to report two small bugs.

The first one. One need to change the code of "select" formatter so that it works without exception in case if usage only formatoptions without having any editoptions defined. The line

var msl = opts.colModel.editoptions.multiple === true ? true : false,

can be modified to

var msl = (opts.colModel.editoptions != null && opts.colModel.editoptions.multiple === true) ? true : false,

for example.

The second bug in View form. It set focus (see the line of focusaref function) on the close elelent of the title (".ui-jqdialog-titlebar-close"), but the current version of jQuery.focus and at least in IE10 ignore it. One can fix the problem for example by adding .attr("tabindex", "-1") in the line or one can add tabindex atribute in the code of createModal (see here).

Best regards
Oleg

19/03/2014
14:03

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello Oleg,

Thanks. Just fixed this in GitHub.

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.

19/03/2014
17:58

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Thank you Tony!

The same problem exist in delGridRow method (see the line). One can fix it in the same way.

Best regards
Oleg

25/03/2014
12:28

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

Thanks again Oleg.

Fixed in GitHub

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.

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
42 Guest(s)

Currently Browsing this Page:
1 Guest(s)