Skip to content

Commit

Permalink
form.js: Ignore buttons again when being asked to render content
Browse files Browse the repository at this point in the history
This slipped through, as I thought the CSS selector `:input`, which
was used previously, is invalid. Although, it's a jQuery specific
selector -.-, which was also previously used.

fixes #5293
  • Loading branch information
nilmerg committed Nov 28, 2024
1 parent 4eadfd0 commit ca2778e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion public/js/icinga/behavior/form.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,9 @@
if ($container[0].contains(origFocus)
&& origFocus.form
&& ! origFocus.matches(
'input[type=submit], input[type=reset], input[type=button], .autofocus, .autosubmit:not(:hover)'
'input[type=submit], input[type=reset], input[type=button]'
+ ', button[type=submit], button[type=reset], button[type=button]'
+ ', .autofocus, .autosubmit:not(:hover)'
)
) {
this.icinga.logger.debug('Not changing content for ' + containerId + ' form has focus');
Expand Down

0 comments on commit ca2778e

Please sign in to comment.