Bug 39524

Summary: App Center - screenshot gallery
Product: UCS Reporter: Alexander Kläser <klaeser>
Component: UMC - App-CenterAssignee: Johannes Keiser <keiser>
Status: CLOSED FIXED QA Contact: Dirk Wiesenthal <wiesenthal>
Severity: enhancement    
Priority: P5 CC: best, gohmann, keiser, klaeser, kramer, wiesenthal
Version: UCS 4.1Keywords: interim-2
Target Milestone: UCS 4.1   
Hardware: Other   
OS: Linux   
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:
Bug Depends on: 38894    
Bug Blocks: 39794    

Description Alexander Kläser univentionstaff 2015-10-14 11:36:17 CEST 
One new feature of the UCS 4.1 App Center is the ability to present thumbnails of multiple screenshots in a gallery/carousel view (cf. attachment 7067 [details]).

+++ This bug was initially created as a clone of Bug #38894 +++

The UMC App Center usability should be improved. The app list is getting longer and longer. :)
Comment 1 Alexander Kläser univentionstaff 2015-10-14 11:36:47 CEST 
(In reply to Alexander Kläser from comment #0)
> One new feature of the UCS 4.1 App Center is the ability to present
> thumbnails of multiple screenshots in a gallery/carousel view (cf.
> attachment 7067 [details]).

Error... attachment 7068 [details].
Comment 2 Johannes Keiser univentionstaff 2015-10-16 12:54:16 CEST 
r 64542

univention-management-console-module-appcenter (5.0.6-24):
Bug #39524:
* Added a Gallery to show multiple images
* Added svg images for the gallery
* Adjusted layout of the 'Details' titlepane slightly

Version: 5.0.6-24.471.201510161251
Comment 3 Johannes Keiser univentionstaff 2015-10-16 14:01:49 CEST 
r 64544

univention-management-console-module-appcenter (5.0.6-25):
* Bug #39524: include svg images

Version: 5.0.6-25.472.201510161303
Comment 4 Johannes Keiser univentionstaff 2015-10-16 14:13:24 CEST 
r 64545

univention-management-console-module-appcenter (5.0.6-26):
* Bug #39524: adjusted a query that gets a domNode for styling

Version: 5.0.6-26.473.201510161408
Comment 5 Johannes Keiser univentionstaff 2015-10-16 15:15:57 CEST 
r 64547

univention-management-console-module-appcenter (5.0.6-27):
* Bug #39524: fixed setting dynamic styles correctly

Version: 5.0.6-27.474.201510161513
Comment 6 Johannes Keiser univentionstaff 2015-10-19 13:28:49 CEST 
univention-management-console-module-appcenter (5.0.6-31):
* Bug #39524: adjusted resizing for gallery images

Version: 5.0.6-31.478.201510191307
Comment 7 Johannes Keiser univentionstaff 2015-10-19 18:06:39 CEST 
r 64602

univention-management-console-module-appcenter (5.0.6-36):
* Bug #39524: Carousel.js cleanup
Comment 8 Johannes Keiser univentionstaff 2015-10-22 19:09:04 CEST 
r univention-appcenter (5.0.6-60):
* Bug #39524: added function to embed youtube videos in the
  AppDetailsPage gallery and added transition for gallery resizing

Version: 5.0.6-60.6.201510221905
Comment 9 Johannes Keiser univentionstaff 2015-10-22 19:20:11 CEST 
r 64777
univention-appcenter (5.0.6-62):
* Bug #39524: added css rules for Carousel transitions

Version: 5.0.6-62.8.201510221917
Comment 10 Dirk Wiesenthal univentionstaff 2015-10-23 15:54:27 CEST 
Don't know if this is related to this bug, but anyway:

domStyle.set(document.querySelectorAll('.umcModule[id^="umc_modules_appcenter"] .umcModuleContent')[0], 'overflow', 'visible')

This breaks the "Installed Apps" module. It also uses AppDetailsPage.js.
Comment 11 Johannes Keiser univentionstaff 2015-10-28 18:21:40 CET 
(In reply to Dirk Wiesenthal from comment #10)
> Don't know if this is related to this bug, but anyway:
> 
> domStyle.set(document.querySelectorAll('.
> umcModule[id^="umc_modules_appcenter"] .umcModuleContent')[0], 'overflow',
> 'visible')
> 
> This breaks the "Installed Apps" module. It also uses AppDetailsPage.js.

r 64949

univention-appcenter (5.0.9-9):
* Bug #39524: Fixed a css selector not working in the installed apps module

Version: 5.0.9-9.28.201510281816
Comment 12 Johannes Keiser univentionstaff 2015-10-30 17:59:45 CET 
(In reply to Johannes Keiser from comment #11)
> (In reply to Dirk Wiesenthal from comment #10)
> > Don't know if this is related to this bug, but anyway:
> > 
> > domStyle.set(document.querySelectorAll('.
> > umcModule[id^="umc_modules_appcenter"] .umcModuleContent')[0], 'overflow',
> > 'visible')
> > 
> > This breaks the "Installed Apps" module. It also uses AppDetailsPage.js.
> 
> r 64949
> 
> univention-appcenter (5.0.9-9):
> * Bug #39524: Fixed a css selector not working in the installed apps module
> 
> Version: 5.0.9-9.28.201510281816

r 65036
univention-appcenter (5.0.14-1):
* Bug #39524: removed setting of domStyle in js code and put
  it in appcenter.styl

Version: 5.0.14-1.39.201510301758
Comment 13 Dirk Wiesenthal univentionstaff 2015-11-02 13:07:46 CET 
I have locally doubled the Thumbnails in samba4.ini, seems to break it:
  http://10.200.4.50/univention-management-console/#module=appcenter:appcenter:0:id:samba4
Comment 14 Johannes Keiser univentionstaff 2015-11-02 18:44:17 CET 
r 65097

univention-appcenter (5.0.14-10):
* Bug #39524: fixed the Carousel widget not displaying correctly when the
  same youtube video is stated more then once

Version: 5.0.14-10.48.201511021842
Comment 15 Dirk Wiesenthal univentionstaff 2015-11-05 12:17:21 CET 
OK. Works, looks good. Changelog entry added
Comment 16 Stefan Gohmann univentionstaff 2015-11-17 12:11:43 CET 
UCS 4.1 has been released:
 https://docs.software-univention.de/release-notes-4.1-0-en.html
 https://docs.software-univention.de/release-notes-4.1-0-de.html

If this error occurs again, please use "Clone This Bug".