Page MenuHomePhabricator
Differential D12187

Polish up Done Button UI States
ClosedPublic
Actions

Authored by chad on Mar 28 2015, 1:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 7:56 PM
Unknown Object (File)
Fri, Dec 13, 9:22 PM
Unknown Object (File)
Thu, Dec 12, 2:28 PM
Unknown Object (File)
Wed, Dec 11, 4:42 PM
Unknown Object (File)
Mon, Dec 9, 7:10 AM
Unknown Object (File)
Sat, Dec 7, 4:46 PM
Unknown Object (File)
Sat, Dec 7, 3:06 PM
Unknown Object (File)
Thu, Dec 5, 2:06 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP637974a1903b: Polish up Done Button UI States
Summary

Improves the UI quite a bit.

  • dashed borders everywhere to denote Unsubmitted
  • $sky sprinkled everywhere to denote Done
  • Consilidate inline-state-is-draft to simply styles.
Test Plan

Sent myself test comments, logged out, read comments on new account. Marked as done, submitted.

pasted_file (151×631 px, 16 KB)

pasted_file (141×648 px, 21 KB)

pasted_file (115×640 px, 13 KB)

pasted_file (128×643 px, 21 KB)

Diff Detail

Repository
rP Phabricator
Branch
inline-ui-revamp-part-2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5016
Build 5034: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 29294.Mar 28 2015, 1:02 AM
chad retitled this revision from to Polish up Done Button UI States.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMar 28 2015, 1:02 AM
chad updated this object.Mar 28 2015, 1:02 AM
chad edited the test plan for this revision. (Show Details)Mar 28 2015, 1:06 AM

This is really feeling pretty good.

epriestley accepted this revision.Mar 28 2015, 1:28 AM
epriestley edited edge metadata.

Putting dashes on the whole thing feels like it might be too heavy to me, and signifying "unsubmitted comment" and "unsubmitted checkmark" in similar ways might be a little confusing, but let's see how it works in practice.

This revision is now accepted and ready to land.Mar 28 2015, 1:28 AM
Closed by commit rP637974a1903b: Polish up Done Button UI States (authored by chad, committed by chad). · Explain WhyMar 28 2015, 1:30 AM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Mar 28 2015, 1:44 AM

itsbeautiful

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
infrastructure/
diff/
view/
55 lines
webroot/
rsrc/
css/
application/
differential/
104 lines

Diff 29294

View Options

resources/celerity/map.php

Loading...
View Options

src/infrastructure/diff/view/PHUIDiffInlineCommentDetailView.php

Loading...
View Options

webroot/rsrc/css/application/differential/phui-inline-comment.css

Loading...
Phabricator · Built by Phacility