Bug 1183741 - User Portal: "Simple" user: Too much vertical space between top-banner and content in case the "Basic | Extended" bar is hidden
Summary: User Portal: "Simple" user: Too much vertical space between top-banner and co...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.UserPortal
Version: ---
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ovirt-4.0.0-beta
: 4.0.0
Assignee: Greg Sheremeta
QA Contact: movciari
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-01-19 16:36 UTC by jniederm
Modified: 2016-07-05 08:03 UTC (History)
11 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-07-05 08:03:10 UTC
oVirt Team: UX
rule-engine: ovirt-4.0.0+
rule-engine: planning_ack+
rule-engine: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)
up-34.png (166.83 KB, image/png)
2015-01-20 14:22 UTC, Pavel Stehlik
no flags Details
up-35.png (171.89 KB, image/png)
2015-01-20 14:23 UTC, Pavel Stehlik
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 57433 0 master MERGED userportal: variable nav bar height 2016-05-16 12:29:51 UTC

Description jniederm 2015-01-19 16:36:39 UTC
Description of problem:
The black "Basic | Extended" bar is not show in userportal if the user is assigned UserRole only. A white space is shown instead.

Version-Release number of selected component (if applicable):
Red Hat Enterprise Virtualization Manager Version: 3.5.0-0.29.el6ev

How reproducible:
100%


Steps to Reproduce:
1. Create a new user with UserRole 
2. Login to userportal using the new user

Actual results:
The bar not visible.


Expected results:
Bar visible

Comment 1 Michal Skrivanek 2015-01-20 08:11:44 UTC
there's a blank white space the size of the bar, which doesn't play nice with the surrounding gray.
we should either save space and move the bottom part up or at least show it in gray

Comment 2 Pavel Stehlik 2015-01-20 14:19:54 UTC
Well, in 3.4 is exactly same positioning for table/cell below it - where are VMs placed. ONly change is the logo & whole bar is more tiny on the top. This cause the white space. Adding scrnshots.

Comment 3 Pavel Stehlik 2015-01-20 14:22:24 UTC
Created attachment 981833 [details]
up-34.png

rhevm 3.4

Comment 4 Pavel Stehlik 2015-01-20 14:23:04 UTC
Created attachment 981834 [details]
up-35.png

rhevm 3.5

Comment 5 Einav Cohen 2015-01-20 16:35:55 UTC
(In reply to Pavel Stehlik from comment #2)
> Well, in 3.4 is exactly same positioning for table/cell below it - where are
> VMs placed. ONly change is the logo & whole bar is more tiny on the top.
> This cause the white space. Adding scrnshots.

correct. 

(In reply to Michal Skrivanek from comment #1)
> there's a blank white space the size of the bar, which doesn't play nice
> with the surrounding gray.
> we should either save space and move the bottom part up or at least show it
> in gray

showing it in gray is problematic, as we don't want "simple" users to be aware that there is an "extended" tab. 
displaying the bar only with the "basic" tab doesn't make much sense. 
the only solution that makes sense here is to save the real-estate, as you suggested, i.e., in case the 'Basic/Extended' tab-panel is hidden, bring the entire main view upwards to be closer to the top-banner. 

@Greg - is this possible?

Comment 6 Greg Sheremeta 2015-01-20 16:46:59 UTC
> @Greg - is this possible?

Yes, should be easy.

Comment 7 Einav Cohen 2015-11-03 18:25:54 UTC
(In reply to Greg Sheremeta from comment #6)
> > @Greg - is this possible?
> 
> Yes, should be easy.

@Greg - is it good material for you to work on with Scott?

Comment 8 Greg Sheremeta 2015-12-16 18:17:19 UTC
I can pair with Scott on this, or he can take it alone probably.

Comment 9 Yaniv Kaul 2016-03-10 15:14:52 UTC
(In reply to Greg Sheremeta from comment #8)
> I can pair with Scott on this, or he can take it alone probably.

Any updates?

Comment 10 Greg Sheremeta 2016-03-10 15:20:17 UTC
No, haven't looked at it.

Comment 11 Sandro Bonazzola 2016-05-02 10:08:45 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 12 Sandro Bonazzola 2016-07-05 08:03:10 UTC
oVirt 4.0.0 has been released, closing current release.


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