Page MenuHomePhabricator

Add a description/toggle to PHUIObjectItemView
ClosedPublic

Authored by chad on May 23 2017, 11:44 PM.
Tags
None
Referenced Files
F15923822: D18006.diff
Mon, Jun 2, 1:59 AM
Unknown Object (File)
Wed, May 14, 1:26 PM
Unknown Object (File)
Sun, May 11, 3:52 PM
Unknown Object (File)
Tue, May 6, 12:34 PM
Unknown Object (File)
Apr 29 2025, 5:32 AM
Unknown Object (File)
Apr 27 2025, 7:03 PM
Unknown Object (File)
Apr 26 2025, 3:43 AM
Unknown Object (File)
Apr 25 2025, 12:11 PM
Subscribers

Details

Summary

Gives the ability to hide a big long block of text in an ObjectListItem without cluttering the UI.

Test Plan

Added a test case to UIExamples. Click on icon, see content. Click again, content go away.

image.png (386×594 px, 31 KB)

image.png (974×2 px, 208 KB)

Diff Detail

Repository
rP Phabricator
Branch
object-item-description (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17221
Build 23054: Run Core Tests
Build 23053: arc lint + arc unit

Event Timeline

  • use in DiffusionHistoryListView
This revision is now accepted and ready to land.May 24 2017, 1:34 PM
src/applications/diffusion/view/DiffusionHistoryListView.php
109

Since this isn't batched, I'd expect it to be quite slow in the general case (e.g., if we show 100 commits and they each mention revisions/users, we'll do 100+ single queries), but we can deal with that later.

(There isn't a very good "right" way to do it right now and I'd like to move us to something like HandlePool since I think that has been successful in making it a lot less cumbersome to do fetches with handles efficiently.)

hmmmmmm, ok, should I file something?

This revision was automatically updated to reflect the committed changes.

Sure, couldn't hurt if you want to jot it down.