|
Joined: Apr 2007
Posts: 3,940 Likes: 1
Former Developer
|
Former Developer
Joined: Apr 2007
Posts: 3,940 Likes: 1 |
showflat (7.1.1) and it's equivalent in 7.2bx still have a <br /> character pumped out by the .tpl regardless of whether there is a Custom title. Since this is a conditional in the .inc.php, why can't it be used by the .tpl as such? This would eliminate extra vertical space above the Avatar and vertical space is cool to not have extra of over there . 2c
|
|
|
|
Joined: Jun 2006
Posts: 16,301 Likes: 116
|
Joined: Jun 2006
Posts: 16,301 Likes: 116 |
An if/else clause would fix this, vs just assuming theres a title. Keep in mind however that Rick's planning to combine the two fields, so no matter what in the next version it will have a value ...
|
|
|
|
Joined: Apr 2007
Posts: 3,940 Likes: 1
Former Developer
|
Former Developer
Joined: Apr 2007
Posts: 3,940 Likes: 1 |
absolutely, i agree.
right now, if you have both it works with 2 br's, but if you only have 1, you still get the 2 br's.
|
|
|
|
Joined: Jun 2006
Posts: 16,301 Likes: 116
|
Joined: Jun 2006
Posts: 16,301 Likes: 116 |
I'm sure it'll be sane in 7.3; since it'll use one or the other, it'll always have something...
|
|
|
Test
by Phun - 05/28/2024 7:31 PM
|
|
|
|
|
|