Page MenuHomePhabricator
Differential D13907

More Ponder Answer polish
ClosedPublic
Actions

Authored by chad on Aug 15 2015, 5:42 PM.
Tags
None
Referenced Files
F14478120: D13907.diff
Sat, Dec 28, 4:35 PM
Unknown Object (File)
Thu, Dec 26, 2:31 AM
Unknown Object (File)
Sun, Dec 22, 1:26 AM
Unknown Object (File)
Tue, Dec 17, 6:32 AM
Unknown Object (File)
Nov 27 2024, 2:14 AM
Unknown Object (File)
Nov 23 2024, 12:56 PM
Unknown Object (File)
Nov 23 2024, 4:34 AM
Unknown Object (File)
Nov 22 2024, 7:34 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