Skip to content

Removing comment around <meta>#2120

Merged
AnmAtAnm merged 1 commit intoRaspberryPiFoundation:developfrom
AnmAtAnm:mobile-meta-bug
Nov 14, 2018
Merged

Removing comment around <meta>#2120
AnmAtAnm merged 1 commit intoRaspberryPiFoundation:developfrom
AnmAtAnm:mobile-meta-bug

Conversation

@AnmAtAnm
Copy link
Contributor

The basics

  • I branched from develop
  • My pull request is against develop
  • My code follows the style guide

The details

Resolves

#2119

Proposed change

Remove comment around .

Reason for Changes

The tag fixes the page scale, and prevents the page from zooming. This keeps the toolbox on screen (can't pan the page if the workspace absorbs the scroll action) and keeps the touch points at reasonable scale.

Test Coverage

Rebuilt mobile demos and tested zoom behavior. The page scale starts correctly, and the page no longer zooms.

Tested on:

  • iOS simulator
  • Chrome mobile preview

Bad commit during debugging.
@AnmAtAnm AnmAtAnm changed the base branch from master to develop November 14, 2018 19:22
@AnmAtAnm
Copy link
Contributor Author

Ignoring Travis, because it is trying to run generator tests, which don't work on Mac (and should have already been corrected). Stranger, it is running the test on master, which is no longer the base of this PR, and does not have generator tests.

@AnmAtAnm AnmAtAnm merged commit 57bffc0 into RaspberryPiFoundation:develop Nov 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants