Page MenuHomePhabricator

Add Action Buttons to ObjectHeaders
ClosedPublic

Authored by chad on Dec 2 2013, 11:33 PM.
Tags
None
Referenced Files
F15410122: D7684.diff
Wed, Mar 19, 6:48 AM
F15401429: D7684.id17349.diff
Mon, Mar 17, 5:39 PM
F15386539: D7684.diff
Sat, Mar 15, 12:48 AM
F15382049: D7684.id17349.diff
Fri, Mar 14, 10:29 AM
Unknown Object (File)
Feb 14 2025, 2:24 AM
Unknown Object (File)
Feb 6 2025, 6:19 AM
Unknown Object (File)
Feb 6 2025, 4:18 AM
Unknown Object (File)
Feb 4 2025, 2:56 PM

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...