prehistoric-systems/keepassxc-browser
1
0
Fork 0
mirror of https://github.com/keepassxreboot/keepassxc-browser.git synced 2026-03-11 08:54:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #892 from keepassxreboot/fix/add_form_div_role_button

Browse source 
Add support for div role button with forms
This commit is contained in:
Sami Vänttinen 2020-06-08 16:17:44 +03:00 committed by GitHub
commit c8cef78fd2
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 15 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
keepassxc-browser/content/keepassxc-browser.js
View file

@ -133,7 +133,18 @@ kpxcForm.onSubmit = async function(e) {
return;
}
const form = this.nodeName === 'FORM' ? this : this.form;
const searchForm = f => {
if (f.nodeName === 'FORM') {
return f;
}
};
// Traverse parents if the form is not found.
const form = this.nodeName === 'FORM' ? this : kpxcFields.traverseParents(this, searchForm, searchForm, () => null);
if (!form) {
return;
}
const usernameId = form.getAttribute('kpxcUsername');
const passwordId = form.getAttribute('kpxcPassword');
const usernameField = _f(usernameId);
@ -1270,6 +1281,8 @@ kpxc.getFormActionUrlFromSingleInput = function(field) {
return action;
};
const formButtonQuery = 'button[type=\'button\'], button[type=\'submit\'], input[type=\'button\'], button:not([type]), div[role=\'button\']';
// Get the form submit button instead if action URL is same as the page itself
kpxc.getFormSubmitButton = function(form) {
const action = kpxc.submitUrl || form.action;
@ -1282,7 +1295,7 @@ kpxc.getFormSubmitButton = function(form) {
}
// Try to find another button. Select the first one.
const buttons = Array.from(form.querySelectorAll('button[type=\'button\'], button[type=\'submit\'], input[type=\'button\'], button:not([type])'));
const buttons = Array.from(form.querySelectorAll(formButtonQuery));
if (buttons.length > 0) {
return buttons[0];
}