Page MenuHomePhabricator

Default newPage() to device-ready
ClosedPublic

Authored by epriestley on Nov 3 2015, 8:48 PM.

Details

Summary

Ref T9690. The "meta viewport" tag got dropped by accident because of the sort of weird logic on the old flow.

Make the default device-ready, then just turn it off for the tiny number of non-device pages.

Test Plan
  • Verified meta viewport tag appears on normal pages again.
  • Verified it doesn't show up on non-mobile pages like Maniphest Reports.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Default newPage() to device-ready.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 3 2015, 8:50 PM
This revision was automatically updated to reflect the committed changes.