Page MenuHomePhabricator

Use hovercards in browse dialog
AbandonedPublic

Authored by epriestley on Jun 20 2016, 8:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 12:19 PM
Unknown Object (File)
Mon, Apr 29, 5:08 PM
Unknown Object (File)
Thu, Apr 25, 2:37 AM
Unknown Object (File)
Fri, Apr 19, 4:56 PM
Unknown Object (File)
Thu, Apr 11, 9:55 AM
Unknown Object (File)
Mon, Apr 8, 2:24 PM
Unknown Object (File)
Apr 8 2024, 8:17 AM
Unknown Object (File)
Apr 4 2024, 2:46 AM
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.