jQuery Grid Plugin - jqGridForum

Topic RSS

setting searchOperators of filterToolbar for column

15/09/2014
11:42

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hi Tony,

the current implementation ofÂ filterToolbar allows to set column operation only on all columns or on no columns usingÂ searchOperators parameter.

On the other side it have no sense for the operationÂ on the column where noÂ searchoptions.sopt is defined. The current implementation just add operation fromÂ defaultSearch in the column and the corresponding context menu don't allows to choose another operation, so the search operator just get some place from searching toolbar:

The demoÂ can be used to reproduce the problem. It would be practical toÂ be able to specifyÂ searchOperators property for the column like in case ofÂ clearSearch. The answerÂ provide a workaround (see the demo), but it would be fine to have the feature out of the box.

Another problem: there are bug/compatibility problem in contextmenu forÂ choosing of searching operation. It works incorrectly in jQuery UI 1.11.0/1.11.0. See the demoÂ which reproduces the problem. Just opens the demo and try to choose another operation for Tax column for example. You will see the problem immediately. The menu looks like

instead of

By the way, I personally find the vertical distance between items in menu too high event in case of "correct" context menu. The context menu from the same demo, but using jQuery UIÂ 1.9.2 (see the demo) looks much better:

Best regards
Oleg

P.S. Sorry, but editing of code/pictures in the forum is really difficult. ItÂ unclear how to write more readable post. :-(

15/09/2014
17:02

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hi Tony,

it seems to me that the CSS rules

.ui-search-menu.ui-menu .ui-menu-item { list-style-image: none; padding-right: 0; padding-left: 0; }
.ui-search-menu.ui-menu .ui-menu-item a { display: block; }
.ui-search-menu.ui-menu .ui-menu-item a.g-menu-item:hover { margin: -1px; font-weight: normal; }

can fix the jQuery UI compatibility problems which I described in the previous post.

Best regards
Oleg

P.S. The pull requestÂ contains the changes.

17/09/2014
09:56

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello Oleg,

Thanks.

I do not think that the first requirement should be added as option (you know too many options) in grid.

As per second - I see it and have just merged the request.

Thanks 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.

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
21 Guest(s)

Currently Browsing this Page:
1 Guest(s)