Bug 36624 - UMC-UVMM module tree + grid height
UMC-UVMM module tree + grid height
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Virtual machines (UVMM)
UCS 4.0
Other Linux
: P5 normal with 2 votes (vote)
: UCS 4.3-2-errata
Assigned To: Johannes Keiser
Ole Schwiegert
:
: 39225 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-13 10:03 CET by Erik Damrose
Modified: 2018-12-05 14:38 CET (History)
4 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.023
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Design, Usability
Max CVSS v3 score:
klaeser: Patch_Available+


Attachments
How it looks (144.09 KB, image/png)
2015-01-21 12:58 CET, Janek Walkenhorst
Details
Screenshot: unused space (104.17 KB, image/png)
2015-10-27 14:23 CET, Philipp Hahn
Details
uvmm_vanished_elements (16.24 KB, image/png)
2018-10-30 14:05 CET, Ole Schwiegert
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Damrose univentionstaff 2014-11-13 10:03:06 CET
The UVMM modules tree has a min-height specified, which leaves large whitespace below the tree if viewed on a small screen / browser with narrow width.
Comment 1 Alexander Kläser univentionstaff 2014-11-13 12:02:42 CET
FYI, the same issue in the LDAP directory browser has been addressed with the following CSS rule:

@media (max-width: 991px) {
    /* force the tree to have a fixed height in UDM modules */
    .umc .umcModuleUDM .dijitTree {
        height: 200px !important;
    }
}
Comment 2 Janek Walkenhorst univentionstaff 2015-01-21 12:58:28 CET
Created attachment 6618 [details]
How it looks
Comment 3 Erik Damrose univentionstaff 2015-01-21 13:10:34 CET
The dual scrollbars - one for the grid, one for the whole page, are not very user friendly. The grid should, as the tree, adjust to the page height.
Comment 4 Alexander Kläser univentionstaff 2015-01-23 11:54:14 CET
(In reply to Janek Walkenhorst from comment #2)
> Created attachment 6618 [details]
> How it looks

(In reply to Erik Damrose from comment #3)
> The dual scrollbars - one for the grid, one for the whole page, are not very
> user friendly. The grid should, as the tree, adjust to the page height.

Now, these two comments refer to the grid height. In the UDM DNS module, we used the CSS class umcDynamicHeight-55 instead of umcDynamicHeight in the following manner:

> domClass.remove(this._grid._grid.domNode, 'umcDynamicHeight');
> domClass.add(this._grid._grid.domNode, 'umcDynamicHeight-55');
Comment 5 Erik Damrose univentionstaff 2015-08-20 09:29:56 CEST
*** Bug 39225 has been marked as a duplicate of this bug. ***
Comment 6 Erik Damrose univentionstaff 2015-08-20 09:30:32 CEST
See https://forge.univention.org/bugzilla/attachment.cgi?id=7122 for a screenshot of the described problem at comment 0
Comment 7 Philipp Hahn univentionstaff 2015-10-27 14:23:25 CET
Created attachment 7227 [details]
Screenshot: unused space

Ugly empty space when window is narrow
Comment 8 Johannes Keiser univentionstaff 2018-10-15 14:10:46 CEST
The Tree and the Grid can now be scrolled individually and the page does not scroll

b2e4827ab0 Bug #36624: Use scrollless searchpage for uvmm as well and move css/logic out of udm
e42950ee5d Bug #36624: Restore scrollposition of grid after update()
b5b48fee71 Bug #36624: Debian changelog entries
85df3ed7a1 Bug #36624: YAML - add entries
44f5a6a99c Bug #36624: Use scrollless searchpage for uvmm as well and move css/logic out of udm
48382804d4 Bug #36624: Restore scrollposition of grid after update()
5b9f6f2f79 Bug #36624: Debian changelog entries
ae8ec0b7fe Bug #36624: YAML - add entries
afdb3e2cb5 Bug #36624: Merge branch 'jkeiser/4.3-2/36624/uvmm_scrollless' into 4.3-2
3194796314 Bug #36624: YAML - add versions



(In reply to Philipp Hahn from comment #7)
> Created attachment 7227 [details]
> Screenshot: unused space
> 
> Ugly empty space when window is narrow
Created bug 47977
Comment 9 Ole Schwiegert univentionstaff 2018-10-30 14:05:14 CET
Created attachment 9720 [details]
uvmm_vanished_elements

I was about to QA this by manually changing the DOM to contain Servers and VM's. But after the update to the testing repo packages I got that. Several elements have vanished/ are not visible anymore
Comment 10 Ole Schwiegert univentionstaff 2018-11-19 11:10:45 CET
The changes work and look good!
Though I still get the problem with the UVMM if I upgrade (uvmm_vanished_elements). Before the changes it is displayed correctly, after upgrade you see vanishing elements.

This is fixed as soon as there is one 'server' available (install KVM)
Comment 11 Ole Schwiegert univentionstaff 2018-11-19 13:54:41 CET
The visual problems seem to be problems unique to a very limited amount of hardware machines, like in the UCS@school installer. Therefore this ticket is done and will be verified

Changelog& Advisory: OK