Page MenuHomePhabricator
Differential D13907

More Ponder Answer polish
ClosedPublic
Actions

Authored by chad on Aug 15 2015, 5:42 PM.
Tags
None
Referenced Files
F18790870: D13907.id33567.diff
Wed, Oct 15, 8:18 PM
F18766762: D13907.id.diff
Tue, Oct 7, 6:23 PM
F18759961: D13907.diff
Mon, Oct 6, 7:05 AM
F18745546: D13907.id.diff
Fri, Oct 3, 7:50 AM
F18741101: D13907.diff
Thu, Oct 2, 8:34 AM
F18712696: D13907.id.diff
Mon, Sep 29, 5:30 AM
F18629864: D13907.diff
Sep 16 2025, 10:06 AM
F18618648: D13907.diff
Sep 15 2025, 12:37 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9099: Redesign Ponder
Commits
Restricted Diffusion Commit
rPfca716d69969: More Ponder Answer polish
Summary

Fixes T9099, I think this is as much as I can come up with for unbeta. Cleans up the answer header (profile image, smaller font, smaller header). Cleans up voting (new, with color), and makes it a bit more readable.

Test Plan

Review a number of answers in ponder with and without votes, comments.

pasted_file (392×718 px, 60 KB)

Diff Detail

Repository
rP Phabricator
Branch
ponder-answer-headers
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7643
Build 8324: [Placeholder Plan] Wait for 30 Seconds
Build 8323: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33567.Aug 15 2015, 5:42 PM
chad retitled this revision from to More Ponder Answer polish.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9099: Redesign Ponder.
Herald added a subscriber: Korvin. · View Herald TranscriptAug 15 2015, 5:42 PM
chad edited the test plan for this revision. (Show Details)Aug 15 2015, 5:43 PM
epriestley accepted this revision.Aug 15 2015, 5:50 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 15 2015, 5:50 PM
Closed by commit rPfca716d69969: More Ponder Answer polish (authored by chad, committed by chad). · Explain WhyAug 15 2015, 5:55 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
applications/
ponder/
controller/
4 lines
view/
35 lines
12 lines
infrastructure/
internationalization/
translation/
1 line
webroot/
rsrc/
css/
application/
ponder/
27 lines

Diff 33567

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionViewController.php

Loading...
View Options

src/applications/ponder/view/PonderAnswerView.php

Loading...
View Options

src/applications/ponder/view/PonderFooterView.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorUSEnglishTranslation.php

Loading...
View Options

webroot/rsrc/css/application/ponder/ponder-view.css

Loading...
Phabricator · Built by Phacility