Bug 1505710 - [UI] - Can't approve operation with 'enter' keyboard button in the setup networks dialog
Summary: [UI] - Can't approve operation with 'enter' keyboard button in the setup netw...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.2.0
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: ovirt-4.2.0
: ---
Assignee: Vojtech Szocs
QA Contact: Pavel Novotny
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-10-24 07:17 UTC by Michael Burman
Modified: 2017-12-20 10:57 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-12-20 10:57:49 UTC
oVirt Team: UX
Embargoed:
rule-engine: ovirt-4.2+
rule-engine: blocker+
rule-engine: planning_ack+
gshereme: devel_ack+
lsvaty: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 83449 0 master MERGED webadmin: Fix dialog key press handling 2017-10-31 20:02:54 UTC

Description Michael Burman 2017-10-24 07:17:04 UTC
Description of problem:
[UI] - Can't approve operation with 'enter' keyboard button in the setup networks dialog.

Pressing the 'OK" button with the 'enter' keyboard in the setup networks dialog does nothing on the latest master. This is a very new regression. 

Version-Release number of selected component (if applicable):
4.2.0-0.0.master.20171023193344.gitc3b23f6.el7.centos

How reproducible:
100%

Steps to Reproduce:
1. Try approve any setup networks operation with 'enter' keyboard button only.

Actual results:
Nothing happens

Expected results:
Must work

Comment 1 Michael Burman 2017-10-24 12:00:51 UTC
The power management configuration approve dialog doesn't work as well with the 'enter' key board when adding host.

Comment 2 Greg Sheremeta 2017-10-24 22:59:16 UTC
@Vojtech, any chance this is related to the recent infra change? https://gerrit.ovirt.org/#/c/82944/

Comment 3 Michael Burman 2017-10-25 08:03:03 UTC
New VM's network interface dialog affected as well

Comment 4 Pavel Novotny 2017-10-30 19:22:16 UTC
This regression seems to affect all dialogs, not just the Network ones.

What I have found out:
- Esc key (dismiss):
    Chromium 62: Does not work at all.
    Firefox 56: Works only when some input widget in the dialog has focus.
- Enter key (submit): Works only when some input widget in the dialog has focus (both browsers).

Comment 5 Red Hat Bugzilla Rules Engine 2017-10-31 12:08:37 UTC
This bug report has Keywords: Regression or TestBlocker.
Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP.

Comment 6 Vojtech Szocs 2017-10-31 15:21:09 UTC
(In reply to Greg Sheremeta from comment #2)
> @Vojtech, any chance this is related to the recent infra change?
> https://gerrit.ovirt.org/#/c/82944/

Yes, looks like that patch caused this regression.

(In reply to Pavel Novotny from comment #4)
> This regression seems to affect all dialogs, not just the Network ones.
> 
> What I have found out:
> - Esc key (dismiss):
>     Chromium 62: Does not work at all.
>     Firefox 56: Works only when some input widget in the dialog has focus.
> - Enter key (submit): Works only when some input widget in the dialog has
> focus (both browsers).

Thanks Pavel for the detailed info.

Looks like OVirtBootstrapModal key press handler code isn't picking up key press events when the dialog (one of its child elements) doesn't have focus. When a dialog is opened but not focused, it doesn't work.

Working on a fix.

Comment 7 Vojtech Szocs 2017-10-31 16:30:23 UTC
Fix verified on following scenarios:

* Hosts / Network Interfaces / Setup Host Networks dialog -> Enter & Escape keys work as expected (regardless if the dialog is focused or not)

* Networks / New dialog -> type some name and press Enter key, new network with given name is created as expected [1]

[1] previously reported as bug 1485927

Comment 8 Michael Burman 2017-11-01 06:05:13 UTC
Hi Vojtech,

Is it going to fix the power management configuration approve dialog when adding host?

Comment 9 Pavel Novotny 2017-11-03 16:25:47 UTC
Verified in 
ovirt-engine-4.2.0-0.0.master.20171102200841.gitb7e2bf5.el7.centos.noarch
ovirt-engine-webadmin-portal-4.2.0-0.0.master.20171102200841.gitb7e2bf5.el7.centos.noarch

The Enter and Esc keys work again as expected.
Verified on dialogs with input elements (New VM, New Network, Edit Host, host Setup Networks)
as well as on dialogs without any (confirming host PM Configuration).
The focus of input elements does not matter.

Tested in browsers:
 Firefox 52.4 ESR
 Firefox 56
 Chromium 62

Comment 10 Vojtech Szocs 2017-11-06 20:51:30 UTC
(In reply to Michael Burman from comment #8)
> Hi Vojtech,
> 
> Is it going to fix the power management configuration approve dialog when
> adding host?

Hi Michael, yes, it's a UI infra fix so it covers all dialogs.

Verification scenarios listed in comment #7 are just two specific instances: first one applies to dialog where no input is focused, second one applies to dialog where some input is focused.

Comment 11 Sandro Bonazzola 2017-12-20 10:57:49 UTC
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.