Remove side spacing on code remarkup
ClosedPublic
Actions

Authored by chad on Feb 9 2015, 6:32 PM.

Details

Reviewers

epriestley
btrahan

Commits

Restricted Diffusion Commit
rPb4d03bb26c61: Remove side spacing on code remarkup

Summary

Sets everything up flush, I think this is the only indented block left.

Test Plan

Write some code.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 28209.Feb 9 2015, 6:32 PM

chad retitled this revision from to Remove side spacing on code remarkup.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added reviewers: epriestley, btrahan.

chad added a subscriber: Design.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptFeb 9 2015, 6:32 PM

chad edited the test plan for this revision. (Show Details)Feb 9 2015, 6:32 PM

test

chad mentioned this in T7166: Putting project to CCs as a condition of Herald Rule doesn't get triggered.Feb 9 2015, 6:45 PM

Not really related, but maybe we should lighten the background colors on these blocks? Their brightness/saturation feels a little out of place to me with the rest of the more nuanced modern design, especially in Diviner/Phriction.

This revision is now accepted and ready to land.Feb 12 2015, 6:08 PM

Yeah, it's something I've noticed as well.

Closed by commit rPb4d03bb26c61: Remove side spacing on code remarkup (authored by chad, committed by chad). · Explain WhyFeb 18 2015, 5:08 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

core/

remarkup.css

8 lines

Diff 28444

View Options

resources/celerity/map.php