Differential D12075

Shorten buttons on Leaving High Security Page
ClosedPublic
Actions

Authored by chad on Mar 14 2015, 5:27 AM.

Details

Reviewers

epriestley

Maniphest Tasks

T7027: "Leaving high security" page looks a bit off on mobile

Commits

Restricted Diffusion Commit
rPaa909ba07213: Shorten buttons on Leaving High Security Page

Summary

Changes the text to just "Stay", which is still obvious what it means, with less copy. Fixes T7027

Test Plan

Now works on mobile.

Diff Detail

Repository

rP Phabricator

Branch

leaving

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4881
Build 4899: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 29051.Mar 14 2015, 5:27 AM

chad retitled this revision from to Shorten buttons on Leaving High Security Page.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

chad added a task: T7027: "Leaving high security" page looks a bit off on mobile.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMar 14 2015, 5:27 AM

epriestley accepted this revision.Mar 14 2015, 11:18 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Mar 14 2015, 11:18 AM

Or maybe just "Cancel".

Closed by commit rPaa909ba07213: Shorten buttons on Leaving High Security Page (authored by chad, committed by chad). · Explain WhyMar 14 2015, 2:35 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

controller/

PhabricatorAuthDowngradeSessionController.php

2 lines

Diff 29051

View Options

src/applications/auth/controller/PhabricatorAuthDowngradeSessionController.php

Shorten buttons on Leaving High Security PageClosedPublicActions