116 posts
  • Has been part of the Envato Community for over 4 years
  • Has collected 1+ items on Envato Market
  • Sells items exclusively on Envato Market
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 :)

2623 posts Put a Donk On It
  • Elite Author: Sold more than $75,000 on Envato Market
  • Sells items exclusively on Envato Market
  • Made it to the Authors' Hall of Fame
  • Has sold $250,000+ on Envato Market
+10 more
ThemeProvince
says

Of course not.

1148 posts
  • Has referred 1+ members
  • Has sold $1,000+ on Envato Market
  • Has been a beta tester for an Envato feature
  • Has collected 10+ items on Envato Market
+3 more
fillerspace
says

Make sure it includes this line:

body {display:none;}

;)

116 posts
  • Has been part of the Envato Community for over 4 years
  • Has collected 1+ items on Envato Market
  • Sells items exclusively on Envato Market
dejans7
says

@thePhoenix thx :)

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

2229 posts
  • Located in Europe
  • Made it to the Authors' Hall of Fame
  • Had an item featured on Envato Market
  • Participated in a focus group or interview to improve the user experience on Envato Market
+11 more
revaxarts
says

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

i prefer

body {text-decoration:blink;}
116 posts
  • Has been part of the Envato Community for over 4 years
  • Has collected 1+ items on Envato Market
  • Sells items exclusively on Envato Market
dejans7
says

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

1035 posts
  • Has referred 500+ members
  • Has sold $500,000+ on Envato Market
  • Has collected 100+ items on Envato Market
  • Elite Author: Sold more than $75,000 on Envato Market
+6 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
  • Has been part of the Envato Community for over 4 years
  • Has collected 1+ items on Envato Market
  • Sells items exclusively on Envato Market
dejans7
says

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

by
by
by
by
by
by