Page MenuHomePhabricator

Remove inline CSS from the Games skin
Open, LowPublic

Description

Inline CSS is generally considered evil, but due to being a legacy skin from 2006 or so, the Games skin still seems to use it in a few places...

  1. Line 1107 (although perhaps this line should be removed altogether and the related logo logic should be revamped and a placeholder logo should be implemented? I dunno.)
  2. Line 1381...
  3. ...1394...
  4. ...and 1407; this is the same as L1381, so we definitely can use a class here to get rid of these three
  5. Line 1416 (again, can't think of any sane reason why this is inline instead of in the main CSS file)