Opened 2 years ago
Closed 2 years ago
#3072 closed Defect (fixed)
Member listing should use elgg_list_entities() in gallery mode for group sidebar.
| Reported by: | brettp | Owned by: | cash |
|---|---|---|---|
| Priority: | normal | Milestone: | Elgg 1.8 Beta |
| Component: | Groups | Version: | Github Master |
| Severity: | trivial | Keywords: | |
| Cc: | brett@… | Difficulty: | easy |
Description
Arrow is above the icon, no hover menu.
Change History (8)
comment:1 Changed 2 years ago by brettp
- Milestone changed from Needs Review to Elgg 1.8 Beta
- Version changed from 1.7 to SVN Trunk
comment:2 Changed 2 years ago by brettp
- Resolution set to fixed
- Status changed from new to closed
comment:3 Changed 2 years ago by cash
I'm almost done with my code for #3045. That should enable us to do elgg_list_entities() rather than this custom for loop. We can pass 'gallery' => true and in my testing everything seems to work as it should.
comment:4 Changed 2 years ago by brettp
- Resolution fixed deleted
- Status changed from closed to reopened
- Summary changed from User hover menus don't display in group sidebar to User hover menus should use elgg_list_entities() in gallery mode for group sidebar.
Alright...that will be perfect. Reopening and slightly changing this ticket.
comment:5 Changed 2 years ago by cash
- Owner set to cash
- Status changed from reopened to new
comment:6 Changed 2 years ago by cash
- Status changed from new to assigned
comment:7 Changed 2 years ago by brettp
- Summary changed from User hover menus should use elgg_list_entities() in gallery mode for group sidebar. to Member listing should use elgg_list_entities() in gallery mode for group sidebar.
comment:8 Changed 2 years ago by cash
- Resolution set to fixed
- Status changed from assigned to closed
(In [svn:8684]) Fixes #3072 using elgg_list_entities*() in group members sidebar view
Note: See
TracTickets for help on using
tickets.
(In [svn:8649]) Fixes #3072: Fixed group member list in sidebar. Thoughts about using the elgg-menu-hz class for this? It's not exactly a menu, but that CSS does exactly what's needed.