Bug 1367895

Summary: ovirt-engine-dashboard layout needs some rework (elements positioning)
Product: [oVirt] ovirt-engine-dashboard Reporter: Pavel Stehlik <pstehlik>
Component: CoreAssignee: Greg Sheremeta <gshereme>
Status: CLOSED WONTFIX QA Contact: Pavel Novotny <pnovotny>
Severity: medium Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: bugs, lsvaty, oourfali, vszocs
Target Milestone: ---Flags: sbonazzo: ovirt-4.2-
Target Release: ---   
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: 2017-08-20 11:17:40 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: UX RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
d-1600.png
none
d-1440.png none

Description Pavel Stehlik 2016-08-17 17:55:42 UTC 
Created attachment 1191704 [details]
d-1600.png

Description of problem:
 This BZ is created as follower for predecessor bug 1347208. Please consider spacing between elements as well as font size. It will come handy not only on large scale environments, but also in situation for monitoring when dashboard is opened in completely separate window. Currently not even 1600x is good alignment. (see also 1440x for the same env). The space between upper boxes is quite generous. Also initial high for boxes should be extended in order to keep same dimensions when line inside are wrapped (eg  Virtual Machines, etc).


Version-Release number of selected component (if applicable):
ovirt-engine-dashboard-1.0.2-1.el7ev.x86_64

How reproducible:
100%

Steps to Reproduce:
1. have some more events in system (or other entities as well)
2. use on lower resolution than FullHD
3.

Actual results:
The boxes are not aligned (they change dimensions/high on more loaded systems).

Expected results:
 The boxes are unified.

Additional info:
Comment 1 Pavel Stehlik 2016-08-17 17:58:18 UTC 
Created attachment 1191706 [details]
d-1440.png

And the smaller res 1440x
Comment 2 Oved Ourfali 2016-08-21 07:43:14 UTC 
Greg, please sit with Serena on these comments, and let's see what/when to fix.
Comment 3 Vojtech Szocs 2016-08-23 16:54:41 UTC 
(In reply to Pavel Stehlik from comment #0)
> It will come handy not only
> on large scale environments, but also in situation for monitoring when
> dashboard is opened in completely separate window.

Since Dashboard UI is implemented as oVirt UI plugin, it's not designed to be opened in a separate browser window.

But maybe I misunderstood and by "completely separate window" you meant another window with WebAdmin UI loaded inside it?

> Also initial high for boxes should be
> extended in order to keep same dimensions when line inside are wrapped (eg 
> Virtual Machines, etc).

PatternFly reference code at [1] uses a jQuery plugin [2] to achieve "all status cards having the same height" appearance. We should use that in Dashboard UI.

[1] http://www.patternfly.org/pattern-library/cards/aggregate-status-card/#/_code

[2] $(stuff).matchHeight();
Comment 4 Greg Sheremeta 2016-08-23 17:51:07 UTC 
@Pavel, re: padding, can you please do some screenshots and highlight in red where you think it can be improved? I've already shaved off as many pixels of whitespace as I can from the original design, and I'm not seeing what you're talking about.

Re: fonts, I'll ask Serena and Kyle, but those are by design. I doubt we want to change them. (We want to go the other way, actually -- move more fonts in oVirt over to the new Open Sans font!)

Re: "Also initial high for boxes should be extended in order to keep same dimensions when line inside are wrapped (eg  Virtual Machines, etc)."

Valid. I opened a separate bug for this, Bug 1369550
Comment 5 Greg Sheremeta 2016-11-30 19:46:59 UTC 
Not going to make 4.1