Bug 39663 - App Center gallery - mouseover with app description is not shown on touch devices
App Center gallery - mouseover with app description is not shown on touch dev...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - App-Center
UCS 4.1
Other Linux
: P5 enhancement (vote)
: UCS 4.1-0-errata
Assigned To: Alexander Kramer
Florian Best
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-10-29 10:08 CET by Alexander Kläser
Modified: 2015-11-26 15:17 CET (History)
3 users (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, Mobile devices/tablets, Usability
Max CVSS v3 score:


Attachments
App tile in normal and hovered state (20.62 KB, image/png)
2015-10-29 10:08 CET, Alexander Kläser
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2015-10-29 10:08:52 CET
Created attachment 7234 [details]
App tile in normal and hovered state

When hovering over an app tile in the UMC App Center gallery, the logo is faded out and the app description appears. However, this is not shown on a touch device. It would be helpful to implement a way for showing the app description on touch devices, as well.

A possible implementation would be to show the app description on the first touch, and open the app detail view on a second touch.
Comment 1 Alexander Kramer univentionstaff 2015-11-19 12:28:08 CET
(In reply to Alexander Kläser from comment #0)
> Created attachment 7234 [details]
> ...
> A possible implementation would be to show the app description on the first
> touch, and open the app detail view on a second touch.

Done. On a touch device [has('touch')] the app description is shown on the first touch and the detail view on the second one.

r 65751
univention-appcenter (5.0.19-6)
Comment 2 Alexander Kramer univentionstaff 2015-11-19 12:41:54 CET
Successful build
Package: univention-appcenter
Version: 5.0.19-6.68.201511191238
Branch: ucs_4.1-0
Scope: errata4.1-0

r 65754 yaml
Comment 3 Florian Best univentionstaff 2015-11-24 13:06:48 CET
yes, it works.

Some technical details:
On non-touch devices every hover of the app toggles a CSS class in the domnode secondTouch. There is a parameter 'false' missing in the domClass.toggle() call. It doesn't break anything.

In general it is not good practice to use the unsanitized app id as HTML id:
1. If the AppCenter wouldn't be a singleton it would break due to duplicated ID's.
2. If the AppCenter would allow characters like !"§$%&/( in the App-Id this would also break.
3. It will still break if we one day release a App with ID == some hardcoded ID we use in the HTML code.
It's better to generate some ID like appcenterOverview-$app_0.

Therefore I tested also an installation of a app which doesn't seem to trigger errors as the reload of the category removes the elements from the DOM first.

YAML: OK
Comment 4 Janek Walkenhorst univentionstaff 2015-11-26 15:17:56 CET
<http://errata.software-univention.de/ucs/4.1/3.html>