Bug 1367895 - ovirt-engine-dashboard layout needs some rework (elements positioning)
Summary: ovirt-engine-dashboard layout needs some rework (elements positioning)
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: ovirt-engine-dashboard
Classification: oVirt
Component: Core
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: ---
Assignee: Greg Sheremeta
QA Contact: Pavel Novotny
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-17 17:55 UTC by Pavel Stehlik
Modified: 2022-06-30 08:13 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-08-20 11:17:40 UTC
oVirt Team: UX
sbonazzo: ovirt-4.2-


Attachments (Terms of Use)
d-1600.png (558.85 KB, image/png)
2016-08-17 17:55 UTC, Pavel Stehlik
no flags Details
d-1440.png (187.67 KB, image/png)
2016-08-17 17:58 UTC, Pavel Stehlik
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1347208 0 unspecified CLOSED Spacing should be changed to better fit the dashboard in 1366x768 resolution 2021-02-22 00:41:40 UTC
Red Hat Issue Tracker RHV-46783 0 None None None 2022-06-30 08:13:19 UTC

Internal Links: 1347208

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


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