Forum

May 22nd, 2014
A A A

Please consider registering
guest

Log In Register

Register | Lost password?
Advanced Search

— Forum Scope —

  

— Match —

   

— Forum Options —

    

Wildcard usage:
*  matches any number of characters    %  matches exactly one character

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

Topic RSS Related Topics
Wrong hovering effect in actions formatter of jqGrid 4.1.0
15/06/2011
10:15
OlegK
Germany
Member
Forum Posts: 1230
Member Since:
10/08/2009
Offline

Hello Tony,

The bug report is based on this my answer on the stackoverflow. The 4.1.0 version of the actions formatter has new hovering effects on the action buttons which works wrong on the line which is in the inline editing mode after the clicking of the "Edit" action button. The hovering shows wrong image from the middle of the corresponding background image:

I suggest to change the action onmouseover for div.ui-inline-save, div.ui-inline-cancel from

jQuery(this).addClass('ui-state-hover');

to

jQuery(this).addClass('ui-state-hover');
jQuery('span',this).removeClass('ui-state-hover'); 

Moreover there are exist another problem if one select another row during a row is in inline mode started with respect of "Edit" action button. The Save and Cancel buttons stay visable:

To fix the problem one have to use the code like

var tr = grid[0].rows.namedItem(lastSel);
$("div.ui-inline-edit, div.ui-inline-del",tr).show();
$("div.ui-inline-save, div.ui-inline-cancel",tr).hide(); 

after calling of restoreRow in the onSelectRow event handle. I don't tested editformbutton mode, but I suppose one will have the same problems in the case.

One should fix the problem or at least introduce new public which should be called after restoreRow for example.

One more remark about the code of the action formatter. One should use jqID method in many places of the code. Morover, like you Tony as good knows, the usage of .rows.namedItem constructs are much more effective as the usage of '#' selector in many places of the current code. Example:

$("tr#"+rid+" div.ui-inline-edit, "+"tr#"+rid+" div.ui-inline-del","#"+gid).show();

Best regards
Oleg

15/06/2011
11:22
tony
Sofia, Bulgaria
Moderator
Forum Posts: 7620
Member Since:
30/10/2007
Offline

Thanks Oleg.

I'm considering to remove these effects.

Tony

For professional UI suites for PHP, ASP.NET and ASP.NET MVC visit us at our commercial products site - trirand.net - by the very same guys that created jqGrid.
15/06/2011
12:31
OlegK
Germany
Member
Forum Posts: 1230
Member Since:
10/08/2009
Offline

Hello Tony,

in general the hovering effects on the icons seems good in my opinion. One should just fix the code. The demo looks prity good. I added here the switching of action buttons in case of the standard editing.

The only thing which I personally not like in the actions formatter code is the usage of "onmouseover" and "onmouseout" instead of unobtrusive style binding. The best way of implementation in my opinion are described here. I mean that it would be good to use additional event handler for onCellSelect or beforeSelectRow events in the code of formatter. I understand that there are one general design problem which need be solved, but one can solve the problem in many way. One should just think a little and decide which way is the best.

Best regards
Oleg 

16/06/2011
10:33
tony
Sofia, Bulgaria
Moderator
Forum Posts: 7620
Member Since:
30/10/2007
Offline

Oleg,

I have fixed the problem. It was really travial copy/paste problem Smile

Not sure that unobtrusive style is applicable in these days.

Imagine which current internet user can work with Java Script disabled Wink

Every solution which work and is usefull for the users is a good solution

I have finished the demos for 4.0 version. You can look here:

http://trirand.com/blog/jqgrid…..qgrid.html

Regards

Tony

For professional UI suites for PHP, ASP.NET and ASP.NET MVC visit us at our commercial products site - trirand.net - by the very same guys that created jqGrid.
16/06/2011
13:59
OlegK
Germany
Member
Forum Posts: 1230
Member Since:
10/08/2009
Offline

Tony,

I find your fix very good (it looks better as my suggestion), but you don't insterted jqID method calls in the lines like

$("tr#"+rid+" div.ui-inline-edit, "+"tr#"+rid+" div.ui-inline-del","#"+gid).show();

or rewrote such lines usaging .rows.namedItem constructs like I mention before.

About reducing of the number of "onmouseover", "onmouseout" and "onclick" handles in the jqGrid I'll write better in the separate "Feature Request" and explain more exactly what I mean.

Best regards
Oleg

P.S. If you prepare the 4.1.1 version, please read another bug report. Currently 1) after clicking on "Reset" button in the Single Search filter dilalog some fields will be visable (all will be afther resetting like in the "Advance Searching" dialog) 2) In both "Single Search" and "Advance Searching" dialog the last rule can be removed which has no sence and which was not permitted in the previous version of the "Advance Searching" dialog. 3) The focus in the Searching dialog is not optimal 4) there are no more keyboard support (Esc or Enter).

16/06/2011
16:58
tony
Sofia, Bulgaria
Moderator
Forum Posts: 7620
Member Since:
30/10/2007
Offline

Oh, Forgot for these. Will do the needed.

Thanks.

For professional UI suites for PHP, ASP.NET and ASP.NET MVC visit us at our commercial products site - trirand.net - by the very same guys that created jqGrid.
17/06/2011
10:29
tony
Sofia, Bulgaria
Moderator
Forum Posts: 7620
Member Since:
30/10/2007
Offline

Oleg,

1) I can not reproduce this. (it work ok for me. If I remember right I have fixed this)

2) Will see

3) What you mean with this? The script (using jqModal) try to focus on the first input field.

4) Esc is supported with closeOnEscape : true in the options (I just checked this and closeOnEscape worked for me) . Enter will be added maybe later.

Kind Regards

Tony

For professional UI suites for PHP, ASP.NET and ASP.NET MVC visit us at our commercial products site - trirand.net - by the very same guys that created jqGrid.
17/06/2011
13:06
OlegK
Germany
Member
Forum Posts: 1230
Member Since:
10/08/2009
Offline

You are right Tony, the most of problems not exist more in the 4.1 version.

The problem with Reset button in the Singe Value Search dialog you can reproduce on the demo, which uses jqGrid 4.0, but the same demo work correct with jqGrid 4.1.

The problem with deleting of the last rule is the following. I mean that it is possible to click on the buttom which I mark red:

After that one receive the dialog

which is misunderstandable for many people.

The setting of focus in the Serching Dialog is probably more the question of the taste or ergonomy. Currently you have it on ADD/OR:

In my opinion the best place would be the last edit field:

See the demo from the answer.

Some additional problem is that it is not so easy to impelment change focus behavior in the new searching dialog because the fields of the dialog fill be repaint. I have to overwrite/chain the reDraw method. It would be nice to have some additional events in the the Filter dialog. For example new events like beforeRedraw and event more important afterRedraw and so on would be very helpfull for the Filter dialog customization.

Best regards
Oleg

13/08/2011
11:02
kobruleht
Member
Forum Posts: 144
Member Since:
05/05/2011
Offline

In 4.1.2 if inline edit is started using edit action button and clicked in other row, old row remains in edit mode.

How to fix this so that inline edit is terminated if other row editing starts or if other row is clicked ?

14/08/2011
09:43
tony
Sofia, Bulgaria
Moderator
Forum Posts: 7620
Member Since:
30/10/2007
Offline
10

Hello,

This is not a bug, but feature and a way the action formatter work.

You can simulate the behaviour without the formatter action – look at the demos.

Kind REgards

For professional UI suites for PHP, ASP.NET and ASP.NET MVC visit us at our commercial products site - trirand.net - by the very same guys that created jqGrid.
14/08/2011
14:30
OlegK
Germany
Member
Forum Posts: 1230
Member Since:
10/08/2009
Offline
11

Hello Tony,

in my opinion kobruleht described the problem not correct. Nevertheless the problem is really exist. I can reformulate the problem so. To be able to implement inline editing jqGrid provide not only the editRow method but the restoreRow method which can be used inside of onSelectRow for example to implement the behavior of inline editing like one needs. The formatter:'actions' hide or make visible the divs with actions. So the usage of restoreRow method along is not enough.

I see the problem so that one should provide public restoreActionIcons method which is close to the internal restorerow method. So one will not need to have the knowladge about the dives names to be able to cancel the row editing. See the answer for details.

Best regards
Oleg 

Forum Timezone: Europe/Sofia

Most Users Ever Online: 215

Currently Online:
38 Guest(s)

Currently Browsing this Page:
1 Guest(s)

Top Posters:

OlegK: 1230

markw65: 179

kobruleht: 144

phicarre: 132

YamilBracho: 124

Renso: 118

Member Stats:

Guest Posters: 447

Members: 11266

Moderators: 2

Admins: 1

Forum Stats:

Groups: 1

Forums: 8

Topics: 10473

Posts: 30981

Newest Members: faez, gadelkareem, info.siva, Krishna, adamf, klimats

Moderators: tony (7620), Rumen[Trirand] (81)

Administrators: admin (61)

Comments are closed.
Privacy Policy   Terms and Conditions   Contact Information