Page MenuHomePhabricator

Add Subscribe capability to hovercard view
Changes PlannedPublic

Authored by AnhNhan on Apr 7 2013, 4:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 17, 11:34 PM
Unknown Object (File)
Sun, Mar 17, 11:25 PM
Unknown Object (File)
Sun, Mar 17, 11:13 PM
Unknown Object (File)
Sun, Mar 17, 11:11 PM
Unknown Object (File)
Feb 13 2024, 6:38 PM
Unknown Object (File)
Dec 7 2023, 10:05 PM
Unknown Object (File)
Nov 26 2023, 1:05 AM
Unknown Object (File)
Oct 30 2023, 3:28 AM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
Restricted Maniphest Task
Summary

Refs T1048 - Subscriptions action added to hovercard by event listener - not for Differential & Maniphest though

Also adds form option to actions

Test Plan

Clicked that button for Pholio Mocks, got subscribed/unsubscribed. Workflow does not work D: (you get a page reload basically)

Diff Detail

Branch
hc_cc
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

src/applications/subscriptions/events/PhabricatorSubscriptionsUIEventListener.php
102

Would probably prefer to have this be 'Subscribed' but have the button inactive, like the action list on the object.

src/applications/subscriptions/events/PhabricatorSubscriptionsUIEventListener.php
102

Oh, thanks for spotting. This wasn't supposed to be here.

Just kicking this back for @chad's comments.

src/applications/subscriptions/events/PhabricatorSubscriptionsUIEventListener.php
94–99

I think this one can go either way, but in general we should err on the side of showing (but disabling/deactivating) options which are possible but not currently permitted -- this lets the user click them to learn why they can't take the action. But just dropping the button in this case seems pretty reasonable.

src/view/widget/hovercard/PhabricatorHovercardView.php
45–53

Maybe this should just be a PhabricatorActionView? Not sure how messy that would be. Would simplify the disabled/inactive stuff too.

AnhNhan edited edge metadata.

Updating this diff, rebasing to latest master. You guys have been very busy.

I'll go for using the action view in hover cards, coming in another diff (soonish).

chad retitled this revision from Adding "CC" capability to hovercard view to Add Subscribe capability to hovercard view.Jun 15 2016, 8:39 PM
chad edited edge metadata.
chad changed the visibility from "All Users" to "Public (No Login Required)".

Oh, this would actually be pretty cool.