#1328 [Opn->Csd]: PATCH: sidebar should be <ul>

ID: 1328 Updated by: helgi@trance.is Reported By: danielc at analysisandsolutions dot com -Status: Open +Status: Closed Package: Web Site Operating System: Irrelevant PHP Version: Irrelevant New Comment: This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. Previous Comments: ------------------------------------------------------------------------ [2004-05-02 18:43:42] helgi at trance dot is just email group@php.net about that issue. ------------------------------------------------------------------------ [2004-05-02 11:23:04] danielc at analysisandsolutions dot com Description: ------------ The sidebar navigation element uses a stream of continuous text separated by <br>'s. These elements are really an unordered list. The <br> configuration causes long method names to appear under the bullet image. Also, the link encompases both the image and the text. Both of these things make the sidebar look really sloppy. Here is a patch that fixes all of these issues. http://www.analysisandsolutions.com/pear/web.sidebar.manual.diff Perhaps it's time to provide me karma for pearweb? Thanks. ------------------------------------------------------------------------ -- Edit this bug report at http://pear.php.net/bugs/bug.php?id=1328&edit=1