jQuery Grid Plugin - jqGridForum

Topic RSS

aria-describedby is a duplicated during addRowData

08/05/2012
23:30

J-Back

New Member

Members

Forum Posts: 2

Member Since:
08/05/2012

Offline

Not sure if this was a new change or not, but t.formatCol returns a aria-describedby attribute which then gets included into an html element. However this element has a manual addition of aria-describedby. innerHTML fails due to the duplicate (removed "aria-describedby=...." from the source and allowed prp to be appended with it which fixed the issue. Seems like this only happend after doing an XML ajax load and then doing an addRowData().

JQuery version 1.7.2

jqGrid version (newest).

Lines affected in jquery.jqGrid.src.js:

2907
2912
2923

-Josh

10/05/2012
14:58

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

Not sure that I understand the problem - especially weith this:

However this element has a manual addition of aria-describedby

Can you explain again and refer the right lines.

Regards

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.

11/05/2012
04:06

J-Back

New Member

Members

Forum Posts: 2

Member Since:
08/05/2012

Offline

My apologies for not being clear.

File: jquery.jqGrid.src.js

Line: 2923

prp = t.formatCol(i,1,v, data, rowid, true);

row[row.length] = "<td role="gridcell" aria-describedby=""+t.p.id+"_"+nm+"" "+prp+">"+v+"</td>";

Notice the manual addition of aria-describedby=... However t.formatCol appends an aria-describedby=... which causes a .innerHTML call in the jQuery method .after() to fail due to the duplicate attribute (tested by removing the additional "aria-describedby" attribute).

11/05/2012
09:52

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Hello,

Thank you very much for this. Fixed 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.

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
58 Guest(s)

Currently Browsing this Page:
1 Guest(s)