BUG/fault themes.inc.php ??? BUG/fault themes.inc.php ???
 

News:

CPG Release 1.6.26
Correct PHP8.2 issues with user and language managers.
Additional fixes for PHP 8.2
Correct PHP8 error with SMF 2.0 bridge.
Correct IPTC supplimental category parsing.
Download and info HERE

Main Menu

BUG/fault themes.inc.php ???

Started by mainpower, June 22, 2006, 08:28:59 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

mainpower

Hello,

If I check the HTML code there are 2 times <b><b> ... therefor I change the following line in themes.inc.php (version 1.4.8 );

<span class="statlink"><b>{STATISTICS}</b></span>

into

<span class="statlink">{STATISTICS}</span>

I guess it's a bug/fault ..... but not sure.

Cheers,
Matthijs  ;D

Joachim Müller

Cosmetical issue, not an actual bug imo. Anyway, moving accordingly...

Paver

I noticed this a while ago and forgot about it.  I'm not sure if I posted about it.

In any case, my thought is that the bold tags should be in the $template variable since that's display and moved out of the $statistics content.  I'll take a look and see if this makes sense.

edit: Nope, it doesn't make sense.  The statistics content is created from a language variable that puts the bold tags in where it likes.  So the original fix is the correct one.

Paver

#3
Fix committed to devel.  I assume it should not be committed to stable since it's only a cosmetic fix?

edit: Oh, I already committed the changes to stable.  I noticed some other cosmetic fixes from May were committed, so is this OK?

Paver

Seems harmless to leave in stable.  Yes, some people may notice that their stats line is no longer all bold.  They can change it if desired in style.css (style 'statlink').

Thread marked as fixed.