Bug 1760001
Summary: | [v2v] Scrollbar container height issues on the migration plan details page | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat CloudForms Management Engine | Reporter: | Mike Turley <mturley> | ||||||
Component: | V2V | Assignee: | Mike Turley <mturley> | ||||||
Status: | CLOSED ERRATA | QA Contact: | Md Nadeem <mnadeem> | ||||||
Severity: | high | Docs Contact: | Red Hat CloudForms Documentation <cloudforms-docs> | ||||||
Priority: | high | ||||||||
Version: | 5.11.0 | CC: | bthurber, dmetzger, fdupont, mnadeem, nperic, simaishi | ||||||
Target Milestone: | GA | Keywords: | ZStream | ||||||
Target Release: | 5.11.6 | Flags: | pm-rhel:
cfme-5.11.z+
|
||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | 5.11.6.0 | Doc Type: | If docs needed, set a value | ||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2020-06-10 13:39:05 UTC | Type: | Bug | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | V2V | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Attachments: |
|
Created attachment 1623847 [details]
Issue reproduced in Chrome
Agh, I typo'd the actual/expected results above. They should be swapped: Actual results: The pagination controls are either unreachable or move past the bottom of the window Expected results: The pagination controls are visible just at the bottom of the window Fabien, somehow my last comment unset the target release and for some reason I don't have a field to change it back.. sorry about that Ah, it was under Advanced. Fixed. Sorry for the noise @Mike, is it still relevant? Fabien, Yes, this is still relevant, but I think we had determined it was low priority. Looks like we never updated the BZ to reflect that. I can fix it for 5.11.6. The above PR#1129 is based on master. Here is an equivalent for ivanchuk: https://github.com/ManageIQ/manageiq-v2v/pull/1130 New commit detected on ManageIQ/manageiq-v2v/jansa: https://github.com/ManageIQ/manageiq-v2v/commit/d162003e511d59cae8425b9d33a1c3111d112c61 commit d162003e511d59cae8425b9d33a1c3111d112c61 Author: Milan Zázrivec <mzazrivec> AuthorDate: Mon May 11 11:55:22 2020 +0000 Commit: Milan Zázrivec <mzazrivec> CommitDate: Mon May 11 11:55:22 2020 +0000 Merge pull request #1129 from mturley/1047-scrollbar-bug [#1047][BZ#1760001] Fix inconsistent scrollbar behavior by removing fixed headers and just having one scroll body for each page (cherry picked from commit 4d797aaade4bea99d1374612a4b97b030deb2a35) https://bugzilla.redhat.com/show_bug.cgi?id=1760001 app/javascript/react/screens/App/App.scss | 14 + app/javascript/react/screens/App/Mappings/Mappings.js | 4 +- app/javascript/react/screens/App/Mappings/components/InfrastructureMappingsList/InfrastructureMappingsList.js | 2 +- app/javascript/react/screens/App/Overview/Overview.js | 54 +- app/javascript/react/screens/App/Plan/Plan.js | 4 +- app/javascript/react/screens/App/Plan/components/PlanRequestDetailList/PlanRequestDetailList.js | 2 +- app/javascript/react/screens/App/Plan/components/PlanVmsList.js | 2 +- app/javascript/react/screens/App/Settings/Settings.js | 4 +- app/javascript/react/screens/App/Settings/__tests__/__snapshots__/Settings.test.js.snap | 6 +- app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsList.js | 2 +- app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/__tests__/__snapshots__/ConversionHostsList.test.js.snap | 8 +- 11 files changed, 54 insertions(+), 48 deletions(-) verified in cfme version: 5.11.6.0.20200526234234 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/RHSA-2020:2480 |
Created attachment 1623846 [details] Issue reproduced in Firefox Description of problem: When viewing the details page of a migration in progress, if there are more VMs than can fit in one page, scrolling the window to the bottom does not work correctly. The problem presents itself differently in different browsers. In Firefox, the page stops scrolling before the bottom, so it is impossible to reach the pagination controls. In Chrome, the page scrolls beyond the bottom, leaving a bunch of white space below the list. See attached screenshots taken in Firefox and Chrome while scrolled as far down as the browser will allow. Version-Release number of selected component (if applicable): How reproducible: 100% Steps to Reproduce: 1. Open a plan details page with a large number if VMs 2. Scroll to the bottom Actual results: The pagination controls are visible just at the bottom of the window Expected results: The pagination controls are either unreachable or move past the bottom of the window Additional info: