Bug 38491 - Change to fixed scrollbar in a UMC module is not smooth enough
Change to fixed scrollbar in a UMC module is not smooth enough
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 4.0
Other Linux
: P5 normal (vote)
: UCS 4.0-2-errata
Assigned To: Johannes Keiser
Alexander Kläser
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-05-07 19:37 CEST by Alexander Kläser
Modified: 2015-08-06 17:54 CEST (History)
1 user (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
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:


Attachments
screenshot of ie10 (win7) (92.61 KB, image/png)
2015-07-28 18:04 CEST, Alexander Kläser
Details
screenshot of chromium with multiple tabs open (160.48 KB, image/png)
2015-07-28 18:16 CEST, Alexander Kläser
Details
Patch (2.84 KB, patch)
2015-07-28 18:28 CEST, Alexander Kläser
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2015-05-07 19:37:29 CEST
When scrolling in a UMC module (e.g., a user detail page), there happens a change when the colored module bar pops up at the top. This change is quite abrupt. A tester would certainly have a better impression with a smoother change at this point.
Comment 1 Alexander Kläser univentionstaff 2015-07-06 14:10:17 CEST
This bug overlaps with Bug 38622.
Comment 2 Johannes Keiser univentionstaff 2015-07-22 15:18:15 CEST
Fixed:

r62301
univention-management-console-frontend (4.1.106-76):
* Bug #38491: smoothened the transition to the fixed ModuleHeader in UMC
        Modules

The ModuleHeader now sticks to the top screen the moment it is reached.

YAML: r62317

r62315

univention-management-console-frontend-theme (0.0.89-19):
* Bug #38491: added css transition for the sticky ModuleHeader

YAML: 62316
Comment 3 Johannes Keiser univentionstaff 2015-07-22 18:08:28 CEST
Merged changes to 4.1-0 branch
Comment 4 Alexander Kläser univentionstaff 2015-07-28 18:04:55 CEST
Created attachment 7062 [details]
screenshot of ie10 (win7)

Looks really smooth, cool :) !

I spotted some with IE10. Open users in 2 separate modules. When scrolling down and up again in the second one, the module header height in the first one is too small.
Comment 5 Alexander Kläser univentionstaff 2015-07-28 18:16:14 CEST
Created attachment 7063 [details]
screenshot of chromium with multiple tabs open

I also managed to provoke the attached situation (in Chromium). To reproduce, I opened a few UDM modules and focused then all opened tabs one after another. I then started to scroll down and then used alt+left / alt+right to jump in the browser history forth and back between the tabs (this new feature has been implemented via Bug 20714). At some point, I ended up with this situation. This is no real blocker, a fix at a later point in time would be IMHO also fine.
Comment 6 Alexander Kläser univentionstaff 2015-07-28 18:28:24 CEST
Created attachment 7064 [details]
Patch

Attached you find some possible clean ups for the ModuleHeader code.
Comment 7 Alexander Kläser univentionstaff 2015-07-29 11:26:38 CEST
The .umcModuleHeaderSticky CSS rule class should have font-size:0.64em in order to finally match the font size of headings in the module.
Comment 8 Johannes Keiser univentionstaff 2015-07-29 12:43:53 CEST
(In reply to Alexander Kläser from comment #7)
> The .umcModuleHeaderSticky CSS rule class should have font-size:0.64em in
> order to finally match the font size of headings in the module.

adjusted:

r62489
univention-management-console-frontend-theme (0.0.89-20):
* Bug #38491: adjusted the font size for the sticky ModuleHeader
  to match the font size from the module page headers
Comment 9 Johannes Keiser univentionstaff 2015-07-29 21:50:04 CEST
(In reply to Alexander Kläser from comment #5)
> Created attachment 7063 [details]
> screenshot of chromium with multiple tabs open
> 
> I also managed to provoke the attached situation (in Chromium). To
> reproduce, I opened a few UDM modules and focused then all opened tabs one
> after another. I then started to scroll down and then used alt+left /
> alt+right to jump in the browser history forth and back between the tabs
> (this new feature has been implemented via Bug 20714). At some point, I
> ended up with this situation. This is no real blocker, a fix at a later
> point in time would be IMHO also fine.

Fixed:

r62509
univention-management-console-frontend (4.1.106-83):
* Bug #38491: adjusted logic for detecting module visibility
Comment 10 Alexander Kläser univentionstaff 2015-07-30 14:13:29 CEST
Changes: OK (in iPad, Android, IE9-11, FF, Chrome)
YAML file: OK
Merge to 4.1: OK

The sticky header does not work in the Android browser, but that problem was present before, as well. This is therefore not regression.

→ VERIFIED
Comment 11 Janek Walkenhorst univentionstaff 2015-08-06 17:51:30 CEST
<http://errata.univention.de/ucs/4.0/268.html>
Comment 12 Janek Walkenhorst univentionstaff 2015-08-06 17:54:30 CEST
<http://errata.univention.de/ucs/4.0/266.html>