Page MenuHomePhabricator

Add Action Buttons to ObjectHeaders
ClosedPublic

Authored by chad on Dec 2 2013, 11:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 27, 1:11 AM
Unknown Object (File)
Fri, Dec 27, 1:11 AM
Unknown Object (File)
Fri, Dec 27, 12:58 AM
Unknown Object (File)
Wed, Dec 25, 10:26 PM
Unknown Object (File)
Tue, Dec 24, 2:18 AM
Unknown Object (File)
Tue, Dec 10, 1:19 PM
Unknown Object (File)
Sat, Dec 7, 5:27 PM
Unknown Object (File)
Nov 27 2024, 10:55 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...