116 posts
  • 5 Years of Membership
  • Collector Level 1
  • Exclusive Author
dejans7
says

Hello, i just want to check this.

Is it a problem if a include a small CSS fix for IE7 only (10 lines of code)? Or a WP theme may be rejected because of that? Can some experienced author or reviewer explain this, please :)

2627 posts Put a Donk On It
  • Weekly Top Seller
  • Elite Author
  • Exclusive Author
  • Featured Author
+11 more
ThemeProvince
says

Of course not.

1149 posts
  • Affiliate Level 1
  • Author Level 3
  • Beta Tester
  • Collector Level 2
+3 more
fillerspace
says

Make sure it includes this line:

body {display:none;}

;)

116 posts
  • 5 Years of Membership
  • Collector Level 1
  • Exclusive Author
dejans7
says

@thePhoenix thx :)

hehe OR this in index.php :)
<!—[if IE]>
DON ’T USE IE , IE IS BAD M ’KAY ?\\?!!
<![endif]—>

2280 posts
  • Top Monthly Author
  • Weekly Top Seller
  • Community Superstar
  • Industrious API Inventor
+15 more
revaxarts
says

Make sure it includes this line: body {display:none;} ;)

i prefer

body {text-decoration:blink;}
116 posts
  • 5 Years of Membership
  • Collector Level 1
  • Exclusive Author
dejans7
says

Oh yeah, blinks go great with IE. I can really imagine IE, blinking text, and a few ├╝ber cool lens flares :) :)

1037 posts
  • Affiliate Level 6
  • Author Level 10
  • Collector Level 4
  • Top Monthly Author
+8 more
ThemeBlvd
says

@thePhoenix thx :) hehe OR this in index.php :)
<!—[if IE]>
DON ’T USE IE , IE IS BAD M ’KAY ?\\?!!
<![endif]—>

It might look nicer to call an external stylesheet even if it’s just for 10 lines of CSS . Another http request might not be the most efficient, but it might look kind of nasty to have all your IE7 fix CSS right there in the head of the theme.

Either way I don’t see why it would make TF reject your item.

116 posts
  • 5 Years of Membership
  • Collector Level 1
  • Exclusive Author
dejans7
says

^ Yeah, i’m calling an external stylesheet, but thanks for the tip anyway :)

by
by
by
by
by
by