Page MenuHomePhabricator

Default newPage() to device-ready
ClosedPublic

Authored by epriestley on Nov 3 2015, 8:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 23, 12:38 PM
Unknown Object (File)
Sun, Mar 17, 11:31 PM
Unknown Object (File)
Sat, Mar 16, 1:53 AM
Unknown Object (File)
Tue, Mar 5, 7:00 PM
Unknown Object (File)
Tue, Mar 5, 7:00 PM
Unknown Object (File)
Tue, Mar 5, 2:33 AM
Unknown Object (File)
Tue, Mar 5, 2:33 AM
Unknown Object (File)
Feb 24 2024, 4:36 PM
Subscribers
None

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.