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

Branch

remarkup-code

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4401
Build 4414: [Placeholder Plan] Wait for 30 Seconds

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

4 lines

Diff 28210

View Options

resources/celerity/map.php