search

ThemeAwesome / WP-Forge

WP-Forge - A Foundation for Sites WordPress Theme
https://themeawesome.com/wp-forge-wordpress-foundation-for-sites-theme/
85 stars 28 forks source link

Header Width Scrolls #40

Closed badcat closed 9 years ago

badcat commented 9 years ago

There's an issue with horizontal scrolling with the

when using a Header image while removing the Site Title and Tagline.

The .header-info class is then collapsing to 0 x 0 px, but the h1 .site-title class still renders a width which make the site exhibit horizontal scrolling. The Header Wrapper is still the default 64rem.

site-title

tsquez commented 9 years ago

I see what you are talking about however I do not experience any horizontal scrolling. I tried it in IE, FF, Safari and Chrome (Windows environment). I do believe that it may be due to the way it is coded to be hidden. When you hide the text in the header the following gets applied:

   .site-title, .site-title a, .site-description {
   clip': rect(1px, 1px, 1px, 1px),
   position: absolute;
}

I am going to correct this in the upcoming version and just apply display:none, combined with text-indent:-625rem;

Thanks for bringing this up :wink: