Page MenuHomePhabricator

Use hovercards in browse dialog
AbandonedPublic

Authored by epriestley on Jun 20 2016, 8:36 PM.
Tags
None
Referenced Files
F14003856: D16154.diff
Sat, Oct 26, 11:34 AM
F13985189: D16154.id38862.diff
Sun, Oct 20, 6:39 PM
Unknown Object (File)
Sep 21 2024, 6:44 AM
Unknown Object (File)
Aug 31 2024, 12:17 PM
Unknown Object (File)
Aug 22 2024, 5:35 PM
Unknown Object (File)
Aug 19 2024, 6:43 AM
Unknown Object (File)
Aug 19 2024, 4:13 AM
Unknown Object (File)
Aug 16 2024, 1:25 PM
Subscribers
None

Details

Summary

Ref T11034. Just proof-of-concept. This doesn't feel completely awful to me, but popping big cards over a small dialog isn't awesome and they can hide the controls, so I don't really love this.

We could possibly pursue this in cojunction with a change that locked the hovercard to the bottom of the dialog, maybe?

+---------------------------+
| (accelerate             ) |
| ...                       |
| ...                       |
| ...                       |
|                [ Cancel ] |
+---------------------------+
  +-----------------------+
  | Hovercard             |
  | Always shows up down  |
  | below here.           |
  +-----------------------+
Test Plan

Screen Shot 2016-06-20 at 1.32.55 PM.png (472×663 px, 61 KB)

Diff Detail

Repository
rP Phabricator
Branch
token1a
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12743
Build 16212: Run Core Tests
Build 16211: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Use hovercards in browse dialog.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.