Page MenuHomePhabricator
Differential D12558

Passphrase credentials should correctly grey out disallowed actions.
ClosedPublic
Actions

Authored by lpriestley on Apr 26 2015, 7:15 PM.
Tags
None
Referenced Files
F18836436: D12558.diff
Mon, Oct 27, 12:56 AM
F18712534: D12558.diff
Sep 29 2025, 5:26 AM
F18296748: D12558.id30146.diff
Aug 23 2025, 4:03 PM
F18041383: D12558.diff
Aug 3 2025, 4:31 AM
F17927237: D12558.id30145.diff
Jul 30 2025, 2:20 PM
Unknown Object (File)
Jun 29 2025, 3:22 AM
Unknown Object (File)
Apr 22 2025, 8:56 PM
Unknown Object (File)
Apr 22 2025, 1:56 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T7493: Buttons are not consistently greyed out in Passphrase
Commits
Restricted Diffusion Commit
rPf347c002a6c6: Passphrase credentials should correctly grey out disallowed actions.
Summary

Fixes T7493, Credential actions in Passphrase should correclty appear greyed out if clicking them will result in a "You Shall Not Pass" dialog.

Test Plan

Create a credential that is visible to everyone but editable by one, login as a user incapable of editing it, navigate to the credential view, all actions should be greyed out, and all actions should open a "You Shall Not Pass" dialog.

Diff Detail

Repository
rP Phabricator
Branch
passphrasegreyoutimpossibleactions
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5514
Build 5533: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 30145.Apr 26 2015, 7:15 PM
lpriestley retitled this revision from to Passphrase credentials should correctly grey out disallowed actions..
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T7493: Buttons are not consistently greyed out in Passphrase.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 26 2015, 7:15 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.Apr 26 2015, 7:17 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 26 2015, 7:17 PM
Closed by commit rPf347c002a6c6: Passphrase credentials should correctly grey out disallowed actions. (authored by lpriestley, committed by epriestley). · Explain WhyApr 26 2015, 7:17 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
passphrase/
controller/
4 lines

Diff 30145

View Options

src/applications/passphrase/controller/PassphraseCredentialViewController.php

Loading...
Phabricator · Built by Phacility