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:
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.
https://github.com/ManageIQ/manageiq-v2v/issues/1047
https://github.com/ManageIQ/manageiq-v2v/pull/1129
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