Page MenuHomeDevCentral

Fix to source code
ClosedPublic

Authored by dereckson on Jul 23 2016, 18:46.
Tags
None
Referenced Files
F45678470: D540.diff
Sun, Aug 23, 03:19
Unknown Object (File)
Thu, Aug 20, 21:58
Unknown Object (File)
Thu, Aug 20, 12:16
Unknown Object (File)
Sun, Aug 16, 19:52
Unknown Object (File)
Sun, Aug 16, 14:51
Unknown Object (File)
Sun, Aug 16, 07:51
Unknown Object (File)
Sun, Aug 2, 12:14
Unknown Object (File)
Sat, Aug 1, 10:04
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.