Univention Bugzilla – Full Text Bug Listing |
Summary: | Offer next/previous buttons in app detail view in order to browse directly to other apps | ||
---|---|---|---|
Product: | UCS | Reporter: | Alexander Kläser <klaeser> |
Component: | UMC - App-Center | Assignee: | Alexander Kramer <kramer> |
Status: | CLOSED FIXED | QA Contact: | Alexander Kläser <klaeser> |
Severity: | enhancement | ||
Priority: | P5 | CC: | gohmann, walkenhorst, wiesenthal |
Version: | UCS 4.1 | ||
Target Milestone: | UCS 4.1-0-errata | ||
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): | Usability | |
Max CVSS v3 score: | |||
Attachments: |
Sketch of different previous/next button positions
Patch for prev and next header buttons Adapted patch fpr prev and next button Improved patch |
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 |
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.