make sure remove buttons do not submit forms
This commit is contained in:
parent
4e8d3d6e11
commit
cb9c47af67
@ -159,6 +159,7 @@ export class MultiSelect extends AbstractSelect<Props, State> {
|
||||
disabled={disabled}
|
||||
aria-disabled={disabled}
|
||||
title={dictionary.removeButtonTitle()}
|
||||
type='button'
|
||||
>
|
||||
<span>
|
||||
<Remove width={20} height={20} />
|
||||
|
@ -121,6 +121,7 @@ export class SingleSelect extends AbstractSelect<Props, State> {
|
||||
disabled={disabled}
|
||||
aria-disabled={disabled}
|
||||
title={dictionary.clearButtonTitle()}
|
||||
type='button'
|
||||
>
|
||||
<span>
|
||||
<Remove width={20} height={20} />
|
||||
|
@ -8,7 +8,7 @@ exports[`MultiSelect renders with empty values 1`] = `
|
||||
'Remove selected values' button. Items can be added by selecting them in
|
||||
the adjacent combobox.</div>
|
||||
<button class="s25-remove s25-offscreen" disabled
|
||||
aria-disabled title="Remove selected values">
|
||||
aria-disabled title="Remove selected values" type="button">
|
||||
<span>
|
||||
<Remove width="20" height="20"></Remove>
|
||||
</span>
|
||||
@ -46,7 +46,8 @@ exports[`MultiSelect renders with values 1`] = `
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<button class="s25-remove" disabled aria-disabled title="Remove selected values">
|
||||
<button class="s25-remove" disabled aria-disabled title="Remove selected values"
|
||||
type="button">
|
||||
<span>
|
||||
<Remove width="20" height="20"></Remove>
|
||||
</span>
|
||||
|
4
dist/select25.js
vendored
4
dist/select25.js
vendored
File diff suppressed because one or more lines are too long
2
dist/select25.js.map
vendored
2
dist/select25.js.map
vendored
File diff suppressed because one or more lines are too long
Loading…
x
Reference in New Issue
Block a user