Hi,

This is a minor issue I had with a recent Framework-based design I made. I post it here as a suggestion.
In page.tpl.php and maintenance-page.tpl.php files, site-name is between strong tags:

            <div id="site-name"><strong>
              <a href="<?php print $front_page; ?>" title="<?php print t('Home'); ?>"><span><?php print $site_name; ?></span></a>
            </strong></div>

When you set font-weight to normal (or 400) for site-name in CSS, the title remains bold in static pages while it changes to normal weight in home page.

I just removed those strong tags in both files:

            <div id="site-name">
              <a href="<?php print $front_page; ?>" title="<?php print t('Home'); ?>"><span><?php print $site_name; ?></span></a>
            </div>

You can adjust font-weight for site-name in your CSS (line 250) for the whole site:

#header #site-name {
(...)
  font-weight: 700;
(...)
}

Comments

andregriffin’s picture

I had placed the site title between <strong> tags to convey extra importance.

Here is an excerpt from HTML5 For Web Designers

The <b> element used to mean, “render this in bold.” Now it is used for some text “to be stylistically offset from the normal prose without conveying any extra importance.” If the text has any extra importance, then the <strong> element would be more appropriate.

Perhaps the CSS should be defaulted to #site-name strong { font-weight: normal; } - I'll have to look into it. But I do agree that there should be an included selector that will globally change the weight, or perhaps be defaulted to a normal weight to begin with.

andregriffin’s picture

Status: Active » Needs review
Pablo Romero’s picture

I finally defaulted normal weight for strong tag in title as you said (#site-name strong { font-weight: 400; }) to preserve the extra importance.
I didn´t think about that because other themes use different tags for the site title (e.g. hgroup). And I had always set a bold text for the title with Framework (until now), which is the most logical setting.
Thanks Andre.

andregriffin’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.