Bug 42229 - Allow grid view of users with photos being visible
Allow grid view of users with photos being visible
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Users
UCS 4.2
Other Linux
: P5 normal (vote)
: UCS 4.2
Assigned To: Jürn Brodersen
Alexander Kläser
: interim-2
Depends on:
Blocks: 43867 44009
  Show dependency treegraph
 
Reported: 2016-09-02 14:33 CEST by Alexander Kläser
Modified: 2017-04-04 18:28 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Feature Request
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
Max CVSS v3 score:


Attachments
Mockup (260.92 KB, image/png)
2016-09-02 14:33 CEST, Alexander Kläser
Details
umc_gallery.png (66.75 KB, image/png)
2017-03-15 06:30 CET, Stefan Gohmann
Details
patch (6.71 KB, patch)
2017-03-16 11:59 CET, Florian Best
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2016-09-02 14:33:41 CEST
Created attachment 7973 [details]
Mockup

See attached mockup, it would be beautiful to have the possibility as in other programs (e.g. Odoo) to have a grid view with photos of all users.
Comment 1 Jürn Brodersen univentionstaff 2017-03-02 10:59:58 CET
univention-management-console-module-udm
r77211

univention-web
r77213


Notes:
* The Bootstrap classes might need adjustment after Bug 43528 and Bug 43521 are fixed.
* I think Bug 43509 is related to this Bug. Sometimes tiles are still visible for a moment after changing to a detail page.
* Encouraging users to set the jpegPhoto might become a problem in big domains: Bug 43694
* Some sort of thumbnail would be nice to reduce bandwidth (either as a cache or generated on demand)
Comment 2 Jürn Brodersen univentionstaff 2017-03-02 18:21:08 CET
r77255 + r77258: cleanup changeView method
Comment 3 Stefan Gohmann univentionstaff 2017-03-15 06:29:28 CET
Very nice feature. See the attached screenshot, I think the size of the elements has to be adjusted.
Comment 4 Stefan Gohmann univentionstaff 2017-03-15 06:30:04 CET
Created attachment 8533 [details]
umc_gallery.png
Comment 5 Florian Best univentionstaff 2017-03-15 11:58:04 CET
I think we should
1. move the search input fields to the top instead of left (is planned via Bug #43528)
2. resize the tiles to 150%
Comment 6 Florian Best univentionstaff 2017-03-16 11:59:39 CET
Created attachment 8553 [details]
patch

Suggestion:
1. Don't query the image via xhr but via a regular GET request.
2. Create a Apache rewrite rule for …/udm/users/user/$dn/image to /udm/image?flavor=...&dn=... → This allows the browser to cache the image
3. Set explicit cache control and expiration HTTP headers to make sure the images are cached for a long time, we could even set Last-Modified from the "modifyTimestamp" LDAP attribute. Generating some E-Tag would also be nice. Maybe we could even add some versioning to the URL, too, by adding the E-Tag into the URL: …/udm/users/user/$dn/$etag/image. Then the cache would immediately be invalidated when the image changes.
Comment 7 Jürn Brodersen univentionstaff 2017-03-17 17:22:52 CET
(In reply to Florian Best from comment #6)
> Created attachment 8553 [details]
> patch
> 
> Suggestion:
> 1. Don't query the image via xhr but via a regular GET request.
> 2. Create a Apache rewrite rule for …/udm/users/user/$dn/image to
> /udm/image?flavor=...&dn=... → This allows the browser to cache the image
> 3. Set explicit cache control and expiration HTTP headers to make sure the
> images are cached for a long time, we could even set Last-Modified from the
> "modifyTimestamp" LDAP attribute. Generating some E-Tag would also be nice.
> Maybe we could even add some versioning to the URL, too, by adding the E-Tag
> into the URL: …/udm/users/user/$dn/$etag/image. Then the cache would
> immediately be invalidated when the image changes.

I will move this into its own bug
Comment 8 Florian Best univentionstaff 2017-03-20 19:47:28 CET
There are no checkboxes. How to select multiple users? (with CTRL).
Comment 9 Jürn Brodersen univentionstaff 2017-03-21 10:38:36 CET
(In reply to Florian Best from comment #8)
> There are no checkboxes. How to select multiple users? (with CTRL).

I think I can add checkboxes (maybe under the menu button?), do you think that feature that would be missed?
Comment 10 Jürn Brodersen univentionstaff 2017-03-21 13:28:42 CET
(In reply to Jürn Brodersen from comment #9)
> (In reply to Florian Best from comment #8)
> > There are no checkboxes. How to select multiple users? (with CTRL).
> 
> I think I can add checkboxes (maybe under the menu button?), do you think
> that feature that would be missed?

See bug 43990
Comment 11 Alexander Kläser univentionstaff 2017-03-29 15:29:01 CEST
Looks good and works fine on mobile and other browsers. I recorded some minor issues at Bug 44129.

→ VERIFIED
Comment 12 Stefan Gohmann univentionstaff 2017-04-04 18:28:42 CEST
UCS 4.2 has been released:
 https://docs.software-univention.de/release-notes-4.2-0-en.html
 https://docs.software-univention.de/release-notes-4.2-0-de.html

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