PEAR is archived and read-only

This mirror preserves historical PEAR package releases and metadata so existing references remain available.

Home » HTML » HTML_QuickForm_Renderer_Tableless » Bug #9115

Suggest removing BR tags

Details

Request #9115Suggest removing BR tags
Submitted2006-10-20 21:35 UTC
Frombenjaminhill at gmail dot com
Assignedwiesemann
StatusClosed
PackageHTML_QuickForm_Renderer_Tableless
PHP Version5.0.0
Roadmaps(Not assigned)

Comments

[2006-10-20 21:35 UTC] benjaminhill at gmail dot com

Description:
------------
My current workaround, to get rid of the BR tags and use just CSS styling, is shown in the test script section.

My feature suggestion is to not have BR line breaks - to let the css take care of it, which increases the flexibility.

As you can see from the code, fixing this is very easy, so overall, low priority feature request.

Test script:
---------------
$form = ...
require_once 'HTML/QuickForm/Renderer/Tableless.php';
$renderer = new HTML_QuickForm_Renderer_Tableless();
$form->accept($renderer);
echo str_replace('<br />','',$renderer->toHtml());

[2006-10-20 22:13 UTC] benjaminhill at gmail dot com

Apologies if the post-rendering BR removal breaks a test case, I was basing it only on my (limited) personal usage of the package, and how the spacing between elements was, in my case, harder to control with the br after the end of each element's div. I don't currently have any code beyond what I've posted. If this reduces compatibility, then no worries.

[2006-11-14 17:01 UTC] benjaminhill at gmail dot com

The new "<LI> instead of <DIV>" solution fixed this issue for me, works great, thanks!