[epiphany/mcatanzaro/passwords: 14/15] ephy.js: Don't require input element name for form autofill
- From: Michael Catanzaro <mcatanzaro src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [epiphany/mcatanzaro/passwords: 14/15] ephy.js: Don't require input element name for form autofill
- Date: Mon, 4 Mar 2019 00:52:31 +0000 (UTC)
commit f944021989f28b23c798d591cc5e6861c31c05ef
Author: Michael Catanzaro <mcatanzaro igalia com>
Date: Sun Mar 3 18:15:44 2019 -0600
ephy.js: Don't require input element name for form autofill
MDN says name is deprecated since HTML *4* so we'd best check for
element ID first, and only fall back to name if missing.
This fixes form autofill for me on at least one website, which is nice.
embed/web-extension/resources/js/ephy.js | 21 ++++++++++++++-------
1 file changed, 14 insertions(+), 7 deletions(-)
---
diff --git a/embed/web-extension/resources/js/ephy.js b/embed/web-extension/resources/js/ephy.js
index d84f7d761..f950956ae 100644
--- a/embed/web-extension/resources/js/ephy.js
+++ b/embed/web-extension/resources/js/ephy.js
@@ -672,17 +672,24 @@ Ephy.FormManager = class FormManager
formAuth.username = null;
if (formAuth.usernameNode && formAuth.usernameNode.value)
- username = formAuth.usernameNode.value;
+ formAuth.username = formAuth.usernameNode.value;
formAuth.usernameField = null;
- if (formAuth.usernameNode && formAuth.usernameNode.name)
- usernameField = formAuth.usernameNode.name;
-
- if (!formAuth.passwordNode.name)
- return null;
+ if (formAuth.usernameNode) {
+ if (formAuth.usernameNode.id)
+ formAuth.usernameField = formAuth.usernameNode.id;
+ else if (formAuth.usernameNode.name)
+ formAuth.usernameField = formAuth.usernameNode.name;
+ }
formAuth.password = formAuth.passwordNode.value;
- formAuth.passwordField = formAuth.passwordNode.name;
+
+ if (formAuth.passwordNode.id)
+ formAuth.passwordField = formAuth.passwordNode.id;
+ else if (formAuth.passwordNode.name)
+ formAuth.passwordField = formAuth.passwordNode.name;
+ else
+ return null;
return formAuth;
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]