Page MenuHomePhabricator

Add Action Buttons to ObjectHeaders
ClosedPublic

Authored by chad on Dec 2 2013, 11:33 PM.
Tags
None
Referenced Files
F15507619: D7684.id.diff
Tue, Apr 15, 9:44 PM
F15477229: D7684.id17349.diff
Mon, Apr 7, 1:04 PM
F15473287: D7684.id17362.diff
Sat, Apr 5, 8:53 PM
F15416702: D7684.id17362.diff
Thu, Mar 20, 1:01 PM
F15410122: D7684.diff
Wed, Mar 19, 6:48 AM
F15401429: D7684.id17349.diff
Mar 17 2025, 5:39 PM
F15386539: D7684.diff
Mar 15 2025, 12:48 AM
F15382049: D7684.id17349.diff
Mar 14 2025, 10:29 AM

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP9942c2a39ee1: Add Action Buttons to ObjectHeaders
Summary

This adds the ability to float action buttons inside ObjectHeaderView.

Test Plan

Tested a UI Example on desktop and mobile. Will test on Notifications next.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

It would maybe be nice for the non-hover state to at least hint that it's a button, maaaaaybe? Not sure if that looks like junk in practice. But this looks similar to action links anyway and doesn't have the same sneakiness to it that the "Mark All Read" link did.

My plan is to just go with it as is on the Notifications page and see how it feels. I've been liking the 'button-less' button look when using Google Analytics, but it may or may not work for us. The current buttons we have are just a bit heavy for the headers, so many gradients...