[richfaces-issues] [JBoss JIRA] Commented: (RF-7138) scrollableDataTable: JavaScript error if user press up/down arrow key in case of only item

[Nagendra krishnawat (JIRA)]

    [ https://jira.jboss.org/jira/browse/RF-7138?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12492952#action_12492952 ] 

Nagendra krishnawat commented on RF-7138:
-----------------------------------------

Priority of this should be major, multi select is very basic requirement of every other application.

> scrollableDataTable: JavaScript error if user press up/down arrow key in case of only item
> ------------------------------------------------------------------------------------------
>
>                 Key: RF-7138
>                 URL: https://jira.jboss.org/jira/browse/RF-7138
>             Project: RichFaces
>          Issue Type: Bug
>          Components: component-ScrollableDataTable
>    Affects Versions: 3.3.0
>         Environment: RichFaces 3.3.0GA
> IE6, IE7, Firefox 3.0.10
>            Reporter: Denis Petrunin
>            Assignee: Konstantin Mishin
>            Priority: Minor
>             Fix For: Future
>
>
> 1) open page with rich:scrollableDataTable, table must contain only item.
> 2) click on item
> 3) press down arrow key
> As result there is JavaScript error:
> this.nTable.rows[1] is undefined
> http://localhost:8080/a4j/g/3_3_0.GAorg/richfaces/renderkit/html/scripts/controls-scrollable-data-table.js
> error in line: 94
> for(var i=0;i<rows.length;i++){rows[i].removeChild(rows[i].cells[index]);}},showRow:function(rowIndex){if(rowIndex=="up"){this.scrollBox.getElement().scrollTop=this.scrollBox.getElement().scrollTop-this.nTable.rows[1].offsetTop;}else if(rowIndex=="down"){this.scrollBox.getElement().scrollTop=this.scrollBox.getElement().scrollTop+this.nTable.rows[1].offsetTop;}else{var row=$(this.gridId+":n:"+rowIndex);var offsetTop=this.nTable.offsetTop+row.offsetTop;if(this.contentBox.getElement().scrollTop>offsetTop){this.scrollBox.getElement().scrollTop=offsetTop;}else{offsetTop+=row.offsetHeight;offsetTop-=this.contentBox.getElement().clientHeight;if(this.contentBox.getElement().scrollTop<offsetTop){this.scrollBox.getElement().scrollTop=offsetTop;}}}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira