Bug 1649057

Summary: Selecting Expression or Scope element causes a loading screen to display twice
Product: Red Hat CloudForms Management Engine Reporter: John Dupuy <jdupuy>
Component: User ExperienceAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: John Dupuy <jdupuy>
Severity: medium Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.10.0CC: bmidwood, dmetzger, mfeifer, simaishi
Target Milestone: GA   
Target Release: 5.10.0   
Hardware: All   
OS: All   
Whiteboard: ux
Fixed In Version: 5.10.0.25 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-02-12 16:51:28 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:

Description John Dupuy 2018-11-12 19:43:44 UTC 
Description of problem:
When creating an expression or scope via the UI, when you select an element, the subsequent dropdown menu loads twice. Between the loading screens, the user cannot select the next dropdown menu. This is true for all Expression or Scope elements, but an example with a condition expression is given below.

Version-Release number of selected component (if applicable):
5.10.0.23.20181106165157_92dd189 (also present in 5.9.6)

How reproducible:
100%

Steps to Reproduce:
1. Navigate to Control->Explorer
2. Select "Conditions" and "Host Conditions" from the Accordion  
3. Create a new condition
4. Under "Edit Selected Element" select an item from the dropdown menu

Actual results:
The next dropdown menu appears after two loading screens

Expected results:
The next dropdown menu appears after one loading screen

Additional info:
After waiting for the second load screen, the user can still create their expression. However, the bug breaks automation.
Comment 2 CFME Bot 2018-11-14 16:02:04 UTC 
https://github.com/ManageIQ/manageiq-ui-classic/pull/4922
Comment 3 CFME Bot 2018-11-16 20:47:28 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/0cca4956f589801912b5cd6e5df4c3bbc109a2e2
commit 0cca4956f589801912b5cd6e5df4c3bbc109a2e2
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Nov 14 10:43:15 2018 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed Nov 14 10:43:15 2018 -0500

    Removed an extra observer from  Edit Selected Element type drop down

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1649057

 app/views/layouts/exp_atom/_editor.html.haml | 3 +-
 1 file changed, 1 insertion(+), 2 deletions(-)
Comment 4 CFME Bot 2018-11-19 14:26:27 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/hammer:

https://github.com/ManageIQ/manageiq-ui-classic/commit/5125ad58a78720bd0c8194cdd049d4722c21b0ff
commit 5125ad58a78720bd0c8194cdd049d4722c21b0ff
Author:     Dan Clarizio <dclarizi>
AuthorDate: Fri Nov 16 15:42:09 2018 -0500
Commit:     Dan Clarizio <dclarizi>
CommitDate: Fri Nov 16 15:42:09 2018 -0500

    Merge pull request #4922 from h-kataria/exp_atom_editor_remove_double_observer

    Removed an extra observer from  Edit Selected Element type drop down

    (cherry picked from commit a126abc9f08ab978669552026424515af7c80130)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1649057

 app/views/layouts/exp_atom/_editor.html.haml | 3 +-
 1 file changed, 1 insertion(+), 2 deletions(-)
Comment 5 John Dupuy 2018-11-26 13:29:46 UTC 
Verified in 5.10.0.25.20181120211723_d2fd659.

Steps for verification:
1. Navigate to Control->Explorer
2. Select "Conditions" and "Host Conditions" from the Accordion  
3. Create a new condition
4. Under "Edit Selected Element" select an item from the dropdown menu

Verified that only one loading screen appears.