Bug 1740055

Summary: [UI] Paginator is misplaced on the Generated Report UI under Optimization.
Product: Red Hat CloudForms Management Engine Reporter: Satyajit Bulage <sbulage>
Component: UI - OPSAssignee: Milan Zázrivec <mzazrivec>
Status: POST --- QA Contact: Sudhir Mallamprabhakara <smallamp>
Severity: low Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: low    
Version: 5.11.0CC: bmidwood, hkataria, lavenel, mshriver, nansari
Target Milestone: GA   
Target Release: 5.11.z   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Screenshot showing actual results.
none
Page per item dropdown bug none

Description Satyajit Bulage 2019-08-12 08:28:05 UTC
Created attachment 1602766 [details]
Screenshot showing actual results.

Description of problem: 


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


How reproducible:100%


Steps to Reproduce:
1. Navigate to Overview --> Optimization
2. Generate any report.
3. Go to Generated report and see Paginator location

Actual results: Paginator is not on the footer side of the page. Also, showing wrong information of Page no's "1-0 of 0"


Expected results: Paginator should be present at the bottom of the page like other page implementations.


Additional info:
The generated report doesn't content any information(empty report).

Comment 2 Parthvi Vala 2019-08-13 12:10:02 UTC
Created attachment 1603295 [details]
Page per item dropdown bug

Comment 3 Parthvi Vala 2019-08-13 12:12:07 UTC
I found another problem with paginator.
See screenshot - https://bugzilla.redhat.com/show_bug.cgi?id=1740055#c2
Look at the 'page per item' dropdown, first few values are not visible at times.

Comment 4 Martin Hradil 2019-08-13 15:57:28 UTC
The original BZ is a "works as intended", we have a new component for saved reports, visible both in Overview > Reports and in Overview > Optimization.


That said, I agree "1-0 of 0" is a bug, there should be a "No records found" instead.
And the same really goes for the "per page" element, it will have enough space when there are records, and it should not be active when there are no records.


(not sure whether this should be 1 BZ or 3)

Comment 6 CFME Bot 2019-08-16 13:27:07 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/6449daf5e089ff61bd2f7c178a11fa07a24320d2
commit 6449daf5e089ff61bd2f7c178a11fa07a24320d2
Author:     Martin Povolny <mpovolny>
AuthorDate: Wed Aug 14 15:33:06 2019 -0400
Commit:     Martin Povolny <mpovolny>
CommitDate: Wed Aug 14 15:33:06 2019 -0400

    Report data: add display of active filter. Add empty state.

    The active filter display allows one click clearing of the filters as
    requested by a BZ.

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

    The display of active filter is partly ready for the multi-criteria
    filtering, once it's available in the API.

    When there are no records, no grid and no paginator is displayed.
    Instead an empty state component with a message is displayed.

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

 app/javascript/components/report-data-table.jsx | 236 +-
 1 file changed, 155 insertions(+), 81 deletions(-)