Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃摉 Clean up wording in doc #16787

Merged
1 commit merged into from Jul 17, 2018
Merged

馃摉 Clean up wording in doc #16787

1 commit merged into from Jul 17, 2018

Conversation

ghost
Copy link

@ghost ghost commented Jul 17, 2018

Cleaned up formatting and wording in spec for amp-pan-zoom.

@ghost ghost requested a review from cathyxz July 17, 2018 12:42
```

## Attributes
##### max-scale (optional)
Specifies a max zoom scale, otherwise set to 3.
Specifies a max zoom scale, which can be a value from 0 - 9. The default value is 3.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From 1 to 9, not from 0 to 9.

@ghost ghost merged commit 72ea125 into ampproject:master Jul 17, 2018
@cathyxz
Copy link
Contributor

cathyxz commented Jul 17, 2018

Actually I should not have approved this. Sorry--the scale from 0 to 9 is incorrect. It makes no sense to have a zoom scale of 0. The min scale is 1.

@ghost
Copy link
Author

ghost commented Jul 17, 2018

I'm about to re-edit it. Checking some of the validator stuff first.

gopanisandip pushed a commit to gopanisandip/amphtml that referenced this pull request Jul 27, 2018
This pull request was closed.
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.

None yet

2 participants