Looks Great to me !
On Monday, June 18, 2012 4:38:09 PM UTC+12, Massimo Di Pierro wrote: > > Check what's in trunk now. Let me know if you think it needs improvement. > > On Sunday, 17 June 2012 15:45:41 UTC-5, Cliff Kachinske wrote: >> >> I agree that the old look, with dividers, *looks* better. >> >> I also think the buttons are easier to find on the page, so I would like >> to optionally use them. >> >> Would you take a patch with the option? >> >> >> On Saturday, June 16, 2012 11:30:04 PM UTC-4, Andrew wrote: >>> >>> A week or two ago a change was made to the nav bar to give the auth >>> buttons for Login, Register, etc a button look. This was part of the >>> fix to the dropdown menus to get them to work with bootstrap. Another >>> change was made to revert back to the old look (but only as an attachment >>> to a previous post in this list). >>> >>> There are two screen shots, current welcome app, and with the buttons >>> reverted to the old look. >>> >>> Credit to Paolo for making both changes. Given that the buttons have >>> been in trunk for a while, it is worth getting a consensus to keep it or >>> revert it. So: >>> >>> *Vote:* >>> *A) KEEP THE BUTTON LOOK* >>> *B) REVERT TO THE OLD LOOK* >>> * >>> * >>> *if B) * >>> * >>> * >>> *B.1) Revert to the Pipe separator between the options (as shown)* >>> *B.2) Don't include the Pipe separator.* >>> >>> >>> B.1) involves updating a recent patch to layout.html: >>> <div id="navbar">{{='auth' in globals() and auth.navbar(separators=(' >>> ',' | ','')) or ''}}</div> >>> >>> A patch for B.1) has already been sent to Massimo (that's my vote :) ) , >>> but let's see what the general feeling is. >>> >>> Thanks >>> >>