Page MenuHomePhabricator

Remove inline CSS from the Games skin
Open, LowPublic


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)

Event Timeline

jack created this task.Jul 2 2016, 12:53 AM