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

馃摉 Update allowed form input elements #22725

Open
wants to merge 1 commit into
base: master
from

Conversation

@westonruter
Copy link
Member

commented Jun 6, 2019

Remove <input type=password> and <input type=file> from disallowed list, since they are now allowed.

See https://amp.dev/documentation/components/amp-form#inputs-and-fields

@googlebot googlebot added the cla: yes label Jun 6, 2019

@cvializ cvializ self-requested a review Aug 8, 2019

@cvializ
cvializ approved these changes Aug 8, 2019
Copy link
Contributor

left a comment

LGTM nice catch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can鈥檛 perform that action at this time.