Home » HTML » HTML_QuickForm_Renderer_Tableless » Bug #9115
Suggest removing BR tags
Details
| Request #9115 | Suggest removing BR tags |
|---|---|
| Submitted | 2006-10-20 21:35 UTC |
| From | benjaminhill at gmail dot com |
| Assigned | wiesemann |
| Status | Closed |
| Package | HTML_QuickForm_Renderer_Tableless |
| PHP Version | 5.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!