Page MenuHomeDevCentral

Fix to source code
ClosedPublic

Authored by dereckson on Jul 23 2016, 18:46.
Tags
None
Referenced Files
F27800067: D540.id1319.diff
Fri, May 8, 07:34
F27772834: D540.diff
Fri, May 8, 02:44
Unknown Object (File)
Tue, May 5, 19:58
Unknown Object (File)
Sun, May 3, 10:03
Unknown Object (File)
Sat, Apr 25, 15:23
Unknown Object (File)
Fri, Apr 24, 17:42
Unknown Object (File)
Tue, Apr 21, 13:22
Unknown Object (File)
Tue, Apr 14, 03:40
Subscribers
None

Details

Summary

A code review shows some minor interesting fixes to do .

  • use "" for charset in SCSS *
  • optimize background CSS declaration *
  • use {{root}} to mark website root in templates
  • add some blank lines

(*) Detected through WebStorm code inspection

Test Plan

Check code correctness in Firefox and Chrome

Diff Detail

Repository
rVIPERWWW Dæghrefn website
Lint
Lint Passed
Unit
No Test Coverage
Branch
improve-source-code
Build Status
Buildable 776
Build 910: arc lint + arc unit

Event Timeline

dereckson retitled this revision from to Fix to source code.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson edited edge metadata.
This revision is now accepted and ready to land.Jul 23 2016, 18:47
This revision was automatically updated to reflect the committed changes.