Bug 832046 - The button which changes to the next grid's page should be grayed out if no objects in the next page
Summary: The button which changes to the next grid's page should be grayed out if no o...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: Einav Cohen
QA Contact:
URL:
Whiteboard: ux
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-06-14 11:39 UTC by Rami Vaknin
Modified: 2016-01-28 21:55 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-23 09:20:27 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)
screenshot (41.68 KB, application/x-gzip)
2012-06-14 11:39 UTC, Rami Vaknin
no flags Details

Description Rami Vaknin 2012-06-14 11:39:18 UTC
Created attachment 591814 [details]
screenshot

Version:
Devel env on Fedora 16, last commit's hash: b151d8a74f9cf012a5828144d2b60fe5df74c5dc

Description:
In case the number of objects in webadmin's main-grid page is as the size of the page (defined by SearchResultsLimit parameter in vdc_options table), the button that moves the grid to the next page is available while it should be grayed out, so it's possible to click on it and move to an empty page.

For instance - if I have 100 vms in the Virtual Machines tab, I can click on the next page button (located in the upper right corner of the Virtual Machines main tab, it will change into page two of the Virtual Machines tab's grid with no objects in it.

Note that this situation happens only when:

the number of objects in main tab's grid % SearchResultsLimit == 0

Comment 1 Einav Cohen 2012-06-14 12:11:55 UTC
are you sure that it is a regression? from what?

Comment 2 Rami Vaknin 2012-06-14 12:14:37 UTC
Pretty sure, from older ovirt-engine commits, I can't point exactly from which commit but I think this regression introduced when the page's objects numbering label first inserted near these buttons not so long ago.

Comment 3 Rami Vaknin 2012-06-14 16:07:06 UTC
Tested it against old code, it seems to happen there too. removing the Regression keyword.

Comment 4 Itamar Heim 2012-12-23 09:20:27 UTC
Closing old bugs. If this issue is still relevant/important in current version, please re-open the bug.


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