Also, RobG's getColumnCells() function works great as a drop-in
replacement for my function, but only in Firefox. IE 7 shows a
javascript error in Prototype's Enumerable class.
On Apr 30, 2:51 pm, Mike van Lammeren <[EMAIL PROTECTED]> wrote:
> Thanks to Justin and RobG for your comments!
>
> To Justin: avoiding selecting every row in the table is a good idea,
> and should be a speed improvement.
>
> To RobG: Why traverse the DOM instead of using the collections? No
> good reason -- I just didn't know about the collections, that's all!
>
> I'm trying to incorporate both of your suggestions, but there is one
> problem I cannot solve. I'm unable to figure out how to extend the
> table collections, so that they recognize PrototypeJS methods.
>
> If we use RobG's suggestion for getRowCells(), then the pluck()
> method, for instance, is no longer recognized.
>
> I have made minor improvements to getRowCells(), and the PrototypeJS
> methods still work:
>
> /**
> * @param element a table element
> * @param integer the requested row
> * @return array an array of table cell elements (<th> or <td>
> tags)
> **/
> getRowCells: function(tableElement, rowId){
> tableElement = $(tableElement); // extend the table element
> var tableRow = tableElement.tBodies[0].rows[rowId];
> if(tableRow)
> {
> return ($(tableRow).descendants()).findAll(function(el){
> return (el.tagName.toUpperCase() == 'TD' ||
> el.tagName.toUpperCase() == 'TH') ? true : false;
> });
> }
> return [];
> }
>
> On Apr 29, 8:40 pm, RobG <[EMAIL PROTECTED]> wrote:
>
> > On Apr 30, 9:13 am, Mike van Lammeren <[EMAIL PROTECTED]> wrote:
>
> > > Here are some handy extensions to select table rows and columns:
>
> > > var TableUtils = {
> > > /**
> > > * @param element a table element
> > > * @param integer the requested row
> > > * @return array an array of table cell elements
> > > **/
> > > getRowCells: function(tableElement, rowIndex){
> > > tableElement = $(tableElement); // extend the table element
> > > var tableRows = tableElement.down().childElements();
> > > if(tableRows[rowIndex])
> > > {
> > > return (tableRows[rowIndex]).descendants();
> > > }
> > > return [];
> > > },
>
> > Why traverse the DOM when there are ready-to-use collections?
>
> > getRowCells: function(tableElement, rowIndex) {
> > return tableElement.rows[rowIndex].cells;
> > }
>
> > > /**
> > > * @param element a table element
> > > * @param integer the requested row
> > > * @return array an array of table cell elements
> > > **/
> > > getColumnCells: function(tableElement, columnIndex){
> > > tableElement = $(tableElement); // extend the table element
> > > var columnCells = [];
> > > var tableRows = tableElement.down().childElements();
> > > tableRows.each(function(trEl, index){ // loop over each table
> > > row
> > > var tCells = trEl.descendants(); // table cells in row
> > > if(tCells[columnIndex])
> > > {
> > > columnCells.push($(tCells[columnIndex])); // extend
> > > each tableElement
> > > }
> > > });
> > > return columnCells;
> > > }}
>
> > getColumnCells: function(tableElement, columnIndex) {
> > var rows = tableElement.rows;
> > var cells = [];
> > for (var i=0, len=rows.length; i<len; i++) {
> > cells.push(rows[i].cells[columnIndex]);
> > }
> > return cells;
> > }
>
> > Incidentally, tableSection elements have a rows collection too, so
> > tableElement may not be the best choice of variable name.
>
> > --
> > Rob
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Spinoffs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/rubyonrails-spinoffs?hl=en
-~----------~----~----~----~------~----~------~--~---
