Page MenuHomePhabricator
Differential D9757

Tighten Remarkup and Fontkit CSS rules in Phabricator
ClosedPublic
Actions

Authored by epriestley on Jun 27 2014, 4:24 PM.
Tags
None
Referenced Files
F13810534: D9757.id23427.diff
Tue, Sep 17, 4:27 PM
F13810533: D9757.id23428.diff
Tue, Sep 17, 4:27 PM
Unknown Object (File)
Thu, Sep 5, 2:55 AM
Unknown Object (File)
Wed, Sep 4, 8:27 PM
Unknown Object (File)
Fri, Aug 30, 12:41 AM
Unknown Object (File)
Fri, Aug 30, 12:40 AM
Unknown Object (File)
Fri, Aug 30, 12:40 AM
Unknown Object (File)
Fri, Aug 30, 12:30 AM
View All Files
Subscribers
epriestley

Details

Reviewers
chad
Maniphest Tasks
Restricted Maniphest Task
T5497: CSS rules are bleeding into widgets in `{Wxxx}` rule
Commits
Restricted Diffusion Commit
rPe42e8aa61875: Tighten Remarkup and Fontkit CSS rules in Phabricator
Summary

Fixes T5497. Scope these down a little bit so they don't bleed into {W...} embeds and such.

Test Plan
  • Viewed a Legalpad document with headers, monospaced stuff, and lists. Looked the same before/after.
  • Viewed a comment with headers, monospace, and lists. Looked the same before/after.
  • Viewed a {W..} embed, now looks sane.

Screen_Shot_2014-06-27_at_9.20.39_AM.png (1×1 px, 181 KB)

Diff Detail

Repository
rP Phabricator
Branch
remstyle
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1431
Build 1431: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 23427.Jun 27 2014, 4:24 PM
epriestley retitled this revision from to Tighten Remarkup and Fontkit CSS rules in Phabricator.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T5497: CSS rules are bleeding into widgets in `{Wxxx}` rule.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 27 2014, 4:24 PM
Harbormaster completed remote builds in B1431: Diff 23427.Jun 27 2014, 4:26 PM
chad accepted this revision.Jun 27 2014, 4:31 PM
chad edited edge metadata.
chad added inline comments.
webroot/rsrc/css/phui/phui-fontkit.css
21

I suppose this could just be one rule.

This revision is now accepted and ready to land.Jun 27 2014, 4:31 PM
epriestley closed this revision.Jun 27 2014, 4:39 PM
epriestley updated this revision to Diff 23428.

Closed by commit rPe42e8aa61875 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
resources/
celerity/
14 lines
src/
infrastructure/
markup/
2 lines
webroot/
rsrc/
css/
core/
52 lines
phui/
10 lines

Diff 23427

View Options

resources/celerity/map.php

Loading...
View Options

src/infrastructure/markup/PhabricatorMarkupEngine.php

Loading...
View Options

webroot/rsrc/css/core/remarkup.css

Loading...
View Options

webroot/rsrc/css/phui/phui-fontkit.css

Loading...
Phabricator · Built by Phacility