Bug #33433 for HTML-Table: delRow

Tue Feb 19 10:20:59 2008 kyle.davenport [...] fmr.com - Ticket created

Subject:

delRow

I need a delRow method. Example of typical usage: I am adding rows one at a time but I need to set attributes on the row _after_ it is added. This means my tests that calculate the flags are happening after addRow, but I may have a test that tells me I don't want the row afterall so that is when I need to delete the last row (the row I just added).

Fri Mar 28 05:57:35 2008 colin.fine [...] pace.co.uk - Correspondence added

On Tue Feb 19 10:20:59 2008, Quickening wrote: Show quoted text

> I need a delRow method. > > Example of typical usage: > > I am adding rows one at a time but I need to set attributes on the row > _after_ it is added. This means my tests that calculate the flags are > happening after addRow, but I may have a test that tells me I don't > want the row afterall so that is when I need to delete the last row > (the row I just added).

As a workround, how about hiding the row? $table->setRowStyle($rowno, 'display:none');

Fri Mar 28 05:57:39 2008 The RT System itself - Status changed from 'new' to 'open'

Fri Mar 28 09:58:42 2008 kyle.davenport [...] fmr.com - Correspondence added

Subject:	RE: [rt.cpan.org #33433] delRow
Date:	Fri, 28 Mar 2008 08:57:29 -0500
To:	<bug-HTML-Table [...] rt.cpan.org>
From:	"Davenport, Kyle" <kyle.davenport [...] fmr.com>

: ) Yes, tried that already! It leaves a row margin or padding, so a bunch of hidden rows look like large blank spaces in the table. Kyle Show quoted text

-----Original Message----- From: Colin Fine via RT [mailto:bug-HTML-Table@rt.cpan.org] Sent: Friday, March 28, 2008 4:58 AM To: Davenport, Kyle Subject: [rt.cpan.org #33433] delRow <URL: http://rt.cpan.org/Ticket/Display.html?id=33433 > On Tue Feb 19 10:20:59 2008, Quickening wrote:

> I need a delRow method. > > Example of typical usage: > > I am adding rows one at a time but I need to set attributes on the row

> _after_ it is added. This means my tests that calculate the flags are

> happening after addRow, but I may have a test that tells me I don't > want the row afterall so that is when I need to delete the last row > (the row I just added).

As a workround, how about hiding the row? $table->setRowStyle($rowno, 'display:none');

Fri Mar 28 12:53:38 2008 a.peacock [...] chime.ucl.ac.uk - Correspondence added

Subject:	Re: [rt.cpan.org #33433] delRow
Date:	Fri, 28 Mar 2008 16:52:50 +0000
To:	bug-HTML-Table [...] rt.cpan.org
From:	Anthony Peacock <a.peacock [...] chime.ucl.ac.uk>

Hi, I haven't forgotten about these. Things are calming down a bit in my day job, so I am hoping to be able to spend some time looking at these requests over the next couple of weeks. Kyle Davenport via RT wrote: Show quoted text

> Queue: HTML-Table > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=33433 > > > : ) Yes, tried that already! It leaves a row margin or padding, so a > bunch of hidden rows look like large blank spaces in the table. > > Kyle > > -----Original Message----- > From: Colin Fine via RT [mailto:bug-HTML-Table@rt.cpan.org] > Sent: Friday, March 28, 2008 4:58 AM > To: Davenport, Kyle > Subject: [rt.cpan.org #33433] delRow > > > <URL: http://rt.cpan.org/Ticket/Display.html?id=33433 > > > On Tue Feb 19 10:20:59 2008, Quickening wrote:

>> I need a delRow method. >> >> Example of typical usage: >> >> I am adding rows one at a time but I need to set attributes on the row

>

>> _after_ it is added. This means my tests that calculate the flags are

>

>> happening after addRow, but I may have a test that tells me I don't >> want the row afterall so that is when I need to delete the last row >> (the row I just added).

> > As a workround, how about hiding the row? > > $table->setRowStyle($rowno, 'display:none'); > > > > >

-- Anthony Peacock CHIME, Royal Free & University College Medical School WWW: http://www.chime.ucl.ac.uk/~rmhiajp/ "I'm in shape. - ROUND is a shape"

Thu Apr 03 10:31:12 2008 colin.fine [...] pace.co.uk - Correspondence added

From:

colin.fine [...] pace.co.uk

On Fri Mar 28 09:58:42 2008, Quickening wrote: Show quoted text

> : ) Yes, tried that already! It leaves a row margin or padding, so a > bunch of hidden rows look like large blank spaces in the table. >

An approach I have used in the past (before I thought of the display trick) was to spread the previous row to cover the one you want to lose, by setCellRowSpan. In fact in my derived class I have a whole load of methods to 'spread' columns and suppress empty columns that way, that I would do a different way now. Colin Fine

Thu Apr 03 12:42:52 2008 kyle.davenport [...] fmr.com - Correspondence added

Subject:	RE: [rt.cpan.org #33433] delRow
Date:	Thu, 3 Apr 2008 11:41:06 -0500
To:	<bug-HTML-Table [...] rt.cpan.org>
From:	"Davenport, Kyle" <kyle.davenport [...] fmr.com>

that might work, but I have some complicated javascript that displays the table in a scrollable window. Don't think it would work if I changed rows sizes. Kyle Show quoted text

-----Original Message----- From: Colin Fine via RT [mailto:bug-HTML-Table@rt.cpan.org] Sent: Thursday, April 03, 2008 9:32 AM To: Davenport, Kyle Subject: [rt.cpan.org #33433] delRow <URL: http://rt.cpan.org/Ticket/Display.html?id=33433 > On Fri Mar 28 09:58:42 2008, Quickening wrote:

> : ) Yes, tried that already! It leaves a row margin or padding, so

a

> bunch of hidden rows look like large blank spaces in the table. >

An approach I have used in the past (before I thought of the display trick) was to spread the previous row to cover the one you want to lose, by setCellRowSpan. In fact in my derived class I have a whole load of methods to 'spread' columns and suppress empty columns that way, that I would do a different way now. Colin Fine

Thu Apr 10 06:17:22 2008 a.peacock [...] chime.ucl.ac.uk - Taken

Thu Apr 10 06:18:19 2008 a.peacock [...] chime.ucl.ac.uk - TimeWorked changed from (no value) to '60'

Thu Apr 10 06:18:19 2008 a.peacock [...] chime.ucl.ac.uk - Status changed from 'open' to 'resolved'

Thu Apr 10 06:18:20 2008 a.peacock [...] chime.ucl.ac.uk - Broken in 2.07b added

Thu Apr 10 06:20:26 2008 a.peacock [...] chime.ucl.ac.uk - Correspondence added 60 min

I have added two new methods: delRow ( row_num ) delSectionRow ( "Section", section_num, row_num ) Passing the value of -1 as row_num will delete the last row. These methods will be in version 2.08 which should be on CPAN later today.

Thu Apr 10 06:20:27 2008 The RT System itself - Status changed from 'resolved' to 'open'

Thu Apr 10 06:21:05 2008 a.peacock [...] chime.ucl.ac.uk - Status changed from 'open' to 'resolved'

Thu Apr 10 10:46:03 2008 kyle.davenport [...] fmr.com - Correspondence added

Subject:	RE: [rt.cpan.org #33433] delRow
Date:	Thu, 10 Apr 2008 09:00:44 -0500
To:	<bug-HTML-Table [...] rt.cpan.org>
From:	"Davenport, Kyle" <kyle.davenport [...] fmr.com>

Thank you, Anthony. That really helps! Kyle Show quoted text

-----Original Message----- From: Anthony Peacock via RT [mailto:bug-HTML-Table@rt.cpan.org] Sent: Thursday, April 10, 2008 5:20 AM To: Davenport, Kyle Subject: [rt.cpan.org #33433] delRow <URL: http://rt.cpan.org/Ticket/Display.html?id=33433 > I have added two new methods: delRow ( row_num ) delSectionRow ( "Section", section_num, row_num ) Passing the value of -1 as row_num will delete the last row. These methods will be in version 2.08 which should be on CPAN later today.

Thu Apr 10 10:46:09 2008 The RT System itself - Status changed from 'resolved' to 'open'

Thu Apr 10 11:23:19 2008 a.peacock [...] chime.ucl.ac.uk - Correspondence added

On Thu Apr 10 10:46:03 2008, Quickening wrote: Show quoted text

> Thank you, Anthony. That really helps!

Version 2.08 has just been uploaded to CPAN, or the tar.gz file can be downloaded from here, if you can't wait for the CPAN mirrors to update: http://www.chime.ucl.ac.uk/~rmhiajp/htmltable/

Thu Apr 10 11:23:40 2008 a.peacock [...] chime.ucl.ac.uk - Status changed from 'open' to 'resolved'