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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Force Fixed Toolbar #2241

Merged
merged 4 commits into from
May 2, 2019
Merged

Conversation

swissspidy
Copy link
Collaborator

This PR enforces the fixed toolbar upon page load and hides the option to change that.

Screenshot of the fixed toolbar:

Screenshot 2019-05-02 at 20 58 25

I've decided against subscribing for any changes here, as I don't think it's needed. I'd rather wait for WordPress/gutenberg#15264.

Fixes #2179.

@googlebot googlebot added the cla: yes Signed the Google CLA label May 2, 2019
@westonruter westonruter merged commit bc2a174 into amp-stories-redux May 2, 2019
@westonruter westonruter added this to the v1.2 milestone May 21, 2019
@swissspidy swissspidy deleted the amp-story/force-top-toolbar branch June 18, 2019 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Signed the Google CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants