Bug 1330580

Summary:	Widgets drag and drop is broken in Reports
Product:	Red Hat CloudForms Management Engine	Reporter:	Dmitry Misharov <dmisharo>
Component:	UI - OPS	Assignee:	Milan Zázrivec <mzazrivec>
Status:	CLOSED ERRATA	QA Contact:	Dmitry Misharov <dmisharo>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5.6.0	CC:	hkataria, jhardy, mpovolny, obarenbo, simaishi
Target Milestone:	GA	Flags:	dmisharo: automate_bug+
Target Release:	5.6.0
Hardware:	All
OS:	All
Whiteboard:	ui:report
Fixed In Version:	5.6.0.5	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-06-29 15:54:55 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Dmitry Misharov 2016-04-26 13:56:26 UTC

Description of problem:
Unable to rearrange widgets in Reports.

Version-Release number of selected component (if applicable):
5.6.0.4-beta2.3.20160421172650_719e256 

How reproducible:
Always

Steps to Reproduce:
1. Navigate to Cloud Intel->Reports
2. Expand Dashboards accordion
3. Got to All Dashboards->Default Dashboard
4. Configuration->Edit this Dashboard
5. Try to rearrange widgets

Actual results:
Unable to rearrange widgets.

Expected results:
Drag&drop has to work with widgets.

Comment 2 Milan Zázrivec 2016-04-27 15:31:45 UTC

https://github.com/ManageIQ/manageiq/pull/8292

Comment 3 CFME Bot 2016-04-29 22:01:04 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/3dd7844f59bd4238fc5f348d946c63289d71dd00

commit 3dd7844f59bd4238fc5f348d946c63289d71dd00
Author:     Milan Zazrivec <mzazrivec>
AuthorDate: Wed Apr 27 17:27:41 2016 +0200
Commit:     Milan Zazrivec <mzazrivec>
CommitDate: Wed Apr 27 17:27:41 2016 +0200

    Bind sortable properties to sortable-handle class
    
    We should not be binding the sortable properties to general elements,
    such as h2, we should rather have our own class to bind the properties
    to.
    
    This is to avoid situations, where drag & drop functionality in our
    UI will stop working whenever we change h2 to h3 or vice versa.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1330580

 app/assets/javascripts/miq_application.js                 | 6 +++---
 app/views/dashboard/_widget.html.haml                     | 2 +-
 app/views/miq_ae_customization/_dialog_resource.html.haml | 2 +-
 app/views/report/_db_widget.html.haml                     | 2 +-
 app/views/report/_widget_form_menu.html.haml              | 4 ++--
 5 files changed, 8 insertions(+), 8 deletions(-)

Comment 4 Dmitry Misharov 2016-05-04 08:16:57 UTC

Verified fixed in 5.6.0.5-beta2.4.20160503153816_1fb554f.

Comment 5 CFME Bot 2016-05-17 17:20:30 UTC

https://github.com/ManageIQ/manageiq/pull/8473

Comment 7 errata-xmlrpc 2016-06-29 15:54:55 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

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

https://access.redhat.com/errata/RHBA-2016:1348