Bug 1656003 - Report Styling: Remove a single style removes the style next to it
Summary: Report Styling: Remove a single style removes the style next to it
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.10.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.11.0
Assignee: Zita Nemeckova
QA Contact: Parthvi Vala
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks: 1672696
TreeView+ depends on / blocked
 
Reported: 2018-12-04 12:42 UTC by Parthvi Vala
Modified: 2019-12-13 15:16 UTC (History)
6 users (show)

Fixed In Version: 5.11.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1672696 (view as bug list)
Environment:
Last Closed: 2019-12-13 15:16:05 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Style Bug Demo (1.66 MB, application/octet-stream)
2018-12-04 12:42 UTC, Parthvi Vala
no flags Details

Description Parthvi Vala 2018-12-04 12:42:23 UTC
Created attachment 1511283 [details]
Style Bug Demo

Description of problem:
While creating or editing a report, if a certain column has more than 1 styles registered, then remove one style, also removes the style immediately after it.
See demo for more understanding.

Version-Release number of selected component (if applicable):
5.10.0.27

How reproducible:
Always

Steps to Reproduce:
1. Navigate to Intel > Reports > Reports(Accordion).
2. Create a new report or edit an existing report, or copy an existing report.
3. Go to the `Styling` tab.
4. Choose a column, and add 3 styles for it.
5. Remove first or second style.

Actual results:
The selected style is removed along with the style after it.

Expected results:
Only the selected style must be deleted.

Additional info:
The issue exists on 5.9 as well, but it used to work fine on 5.8.

Comment 5 CFME Bot 2018-12-18 14:46:44 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/fd80afa47a39a574e5d195c490d51ee754e6f18a
commit fd80afa47a39a574e5d195c490d51ee754e6f18a
Author:     Zita Nemeckova <znemecko>
AuthorDate: Tue Dec 18 04:47:29 2018 -0500
Commit:     Zita Nemeckova <znemecko>
CommitDate: Tue Dec 18 04:47:29 2018 -0500

    Use onchange only for radio buttons

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

 app/assets/javascripts/miq_ujs_bindings.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 7 Parthvi Vala 2019-05-02 09:16:35 UTC
FIXED. Verified on 5.11.0.2.20190430174828_0e34dea.


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