Long site title doesn't fit in header area

[DJPaul]

Long Site Title - "The Big Brown Fox Jumps Over the Lazy Dog"

The beginning of the title is pushed up to behind the BP Admin Bar.

Review feedback: "This can be addressed by adding documentation in readme.txt file about the limitations for the site title and/or how to allow for long site title using the theme by e.g. sharing CSS solution to allow for long site title."

It would be preferable to see if there are any tweaks that can be made to HTML/CSS to accommodate this.

[karmatosed]

Attached suggestion I changed the spacing at bottom and also line height. I've also included a screenshot of what happens to the header now both with a long and short title.

[karmatosed]

Passing to you Paul for confirmation.

[hnla]

Just as a foot note to this issue. the reason - as Tammie has alluded to - for this issue is the use of a percentage value for line-height and in understanding why that is generally not an advisable approach, generally line-height should be expressed as a commuted value e.g '1.3'

I note that the body font has also been set as a percentage value and if it hasn't ought to be changed.

[DJPaul]

@hnla Where do we set any font as a %?

[DJPaul]

3234-2.patch is a different approach; I've taken out the line-height et al, and the absolute positioning from that and the search form elements. I have assumed the design intention was to try to get the title and the search form on the same vertical level, so this patch also improves that and general positioning. Feedback and testing appreciated :)

[hnla]

I was referring to the shorthand font property declaration on body the line-height has been set as % it should not be really no line-heights should be set as % or em else compound inheritance issues will follow. Remove the line-height from the heading tag, change body to something like 1.3 unit-less (value is then computed from the actual element inheriting that line-height

I can tell that the original intent was to use line-height as a positioning property which it is not and should not be used as it's the leading value for inline text, margins position/space block elements.

[hnla]

In looking at the ticket for comment spacing I've noticed that we also have an issue with the activity meta header and looong group names see attached SC(or any long names I guess) I'll add a fix for this on the patch for the comment spacing but it has consequences.

[hnla]

SC showing view.delete links vanishing behind fav button

[hnla]

SC showing margin-right margin to clear fav button

[Dennissmolek]

@hnla, dont forget the reply button if you are a member. I was testing it and 130px seems to work for me.

[hnla]

Yep did take into account the reply button just after taking the SC which was really just to illustrate the issue.

[djpaul]

(In [4418]) Fit a longer site title into the page header. Fixes #3234, props karmatosed for initial patch