Page MenuHomePhabricator

Better mobile display of ObjectItemView
ClosedPublic

Authored by chad on Apr 10 2014, 4:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 15, 3:49 PM
Unknown Object (File)
Thu, Apr 11, 1:06 PM
Unknown Object (File)
Sat, Mar 30, 1:08 AM
Unknown Object (File)
Mar 23 2024, 5:09 AM
Unknown Object (File)
Mar 20 2024, 11:35 PM
Unknown Object (File)
Feb 13 2024, 11:41 PM
Unknown Object (File)
Feb 13 2024, 11:41 PM
Unknown Object (File)
Feb 13 2024, 11:41 PM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP3018a5eb0912: Better mobile display of ObjectItemView
Summary

Makes the default 3 rows on mobile devices, with larger fonts. Differential/Audit is much better, Maniphest is maybe a sidegrade depending on setup.

Test Plan

test maniphest, audit, differential list items.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

chad retitled this revision from to Better mobile display of ObjectItemView.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

One thought I've had about titles getting truncated is that we could put a little transparent-to-grey gradient over the last, say, 12 pixels of them, so the right hand side would fade out over the span of a character or so, instead of cutting off abruptly. I think the current behavior overall is fine, but it looks a little unintentional right now when we cut abruptly in the middle of a character, and right at the edge.

Other simpler fixes might be:

  • Cut abruptly, but leave a few pixels of border between the cut position and the edge, so it's more clear that the cut is intentional?
  • Use "text-overflow: ellipsis" -- we'll lose a bit of information and I'm not sure this is supported everywhere yet, and some of the positioning tricks might make it messy, but it would look clearly intentional?
This revision is now accepted and ready to land.Apr 10 2014, 2:37 PM

Yeah, this isn't perfect. With text-overflow you still need to know the width I think. I considered just removing the icon, but went this way instead which seemed slightly better.

The icon one looks good to me in the new stuff, it's just this that still looks like it's not intentional, since it's smushing right into the edge there:

Screen_Shot_2014-04-10_at_7.45.48_AM.png (141×209 px, 9 KB)

oh that, yeah... let me see what solutions there may be. It's not a regression at least, just always been slightly jank.

Yeah -- this is super minor, it has just been bugging me for a while.

chad updated this revision to Diff 20746.

Closed by commit rP3018a5eb0912 (authored by @chad).