Univention Bugzilla – Bug 39664
Offer next/previous buttons in app detail view in order to browse directly to other apps
Last modified: 2016-02-04 14:10:54 CET
Created attachment 7235 [details] Sketch of different previous/next button positions If browsing through the different apps in the UMC App Center, one needs currently several mouse clicks to switch from one app detail view to the next one. It would be thus helpful to show previous/next buttons in the detail page to allow for a quick switch to consecutive apps. The implementation could be done as buttons in the module header or with arrow buttons on the left/right side of the app page, cf. attached sketch.
Created attachment 7315 [details] Patch for prev and next header buttons IMHO we should prefer the header button solution: 1) The icon to close the DetailPage is already inside the header, so all other control elements should also appear there. 2) The header with the buttons is always visible to the user. You don't have to scroll to switch the app. 3) The carousel already has similar icons as the second suggestion, I am afraid that this could confuse the user. This could be a case for ab-testing.
univention-appcenter (5.0.19-8) * Bug #39664: Offer next/previous buttons in app detail view Successful build Package: univention-appcenter Version: 5.0.19-8.71.201511250901 User: akramer Branch: ucs_4.1-0 Scope: errata4.1-0 Successful build Package: univention-appcenter Version: 5.0.19-8.71.201511250901 Branch: ucs_4.1-0 Scope: errata4.1-0 r 65885 yaml file
Actually looks cool. But the DetailPage flickers when hitting next/previous. Maybe one should render a new DetailPage in the background and then add an Animation from the left/right so that it is clear what just happened. Anyway, please revert for today
r 65900 As discussed with Dirk, I reverted the changes from 65884. Successful build Package: univention-appcenter Version: 5.0.19-10.73.201511251502 Branch: ucs_4.1-0 Scope: errata4.1-0 r 65902 also revert yaml file
(In reply to Dirk Wiesenthal from comment #3) > Actually looks cool. But the DetailPage flickers when hitting next/previous. > Maybe one should render a new DetailPage in the background and then add an > Animation from the left/right so that it is clear what just happened. Animation would be nice, however, I do not think that the extra effort needed would pay off. @Alex: Cool :) ! I would suggest to store the current list of visible app IDs at the level of the AppDetailPage and not the apps themselves.
Created attachment 7328 [details] Adapted patch fpr prev and next button As requested I adapted the patch.
(In reply to Alexander Kramer from comment #6) > Created attachment 7328 [details] > Adapted patch fpr prev and next button > > As requested I adapted the patch. I like it like this :) . A little suggestion to avoid "undefined" being used as currentIndex: > var currentIndex = -1; > array.some(this.visibleApps, lang.hitch(this, function(iapp, idx) { > if (iapp.id == this.app.id) { > currentIndex = idx; > return true; > } > }));
Created attachment 7349 [details] Improved patch As requested I added a initial value of -1 for currentIndex
r66251 Applied patch Successful build Package: univention-appcenter Version: 5.0.19-21.80.201512101233 Branch: ucs_4.1-0 Scope: errata4.1-0 r66252 yaml file
Looks great :) ! Change: OK YAML file: OK (minor adjustments of the bug description) → VERIFIED
<http://errata.software-univention.de/ucs/4.1/79.html>