Bug #13994
closedSome Site icons missing on Site archive
Added by Sonja Leix over 3 years ago. Updated almost 3 years ago.
0%
Description
I'm seeing a lot of the site icons missing images and displaying overflow meta text. See screenshot.
Here is an example: https://commons.gc.cuny.edu/sites/?filters%5Bkeyword%5D=english&order-by=active&bpage=2
I wonder if we need to define a fallback image to prevent this.
Files
Screen Shot 2021-02-16 at 12.01.03 PM.png (267 KB) Screen Shot 2021-02-16 at 12.01.03 PM.png | Sonja Leix, 2021-02-16 02:01 PM |
Related issues
Updated by Boone Gorges over 3 years ago
- Assignee set to Raymond Hoh
- Target version set to 1.18.5
This is likely related to the change in #1173. Ray, could you have a look, and ensure we fall back on something that makes sense?
Updated by Raymond Hoh over 3 years ago
- Category name set to BuddyPress (misc)
- Status changed from New to In Progress
There were some changes to the site icon in BuddyPress 7.0, which affected us.
See https://bpdevel.wordpress.com/2020/11/01/meet-the-new-default-avatar-for-sites/.
I'm looking into it.
Updated by Raymond Hoh over 3 years ago
In BuddyPress 7.0, a new, default site icon was introduced. This icon is used if one isn't set from the site's admin dashboard.
It looks like this:
We have some custom code that broke this display (also found a bug in BuddyPress!). Do we want this to be the default site icon? It's a little bland.
Or do we want to revert to the previous, random retro
, arcade-inspired icons. Here's one example:
Similarly, BuddyPress introduced a default group icon back in v2.6, but we're not using it.
It looks like this:
We chose to use the retro
icons instead. See #1625. Maybe it's worth revisiting our icon defaults with a fresh set of eyes.
Updated by Sonja Leix over 3 years ago
Thanks Ray for looking into this. I'd say let's use the new default icon (grey). The redesign will be a lot cleaner and more modern, I'm not sure if the retro really still works and we might overall want to phase it out. Using the grey icon might perhaps also be an incentive for site owners to add a custom site icon to stand out.
Updated by Boone Gorges over 3 years ago
+1 to the grey icons. The retro ones are too busy for my taste. But I recall that Matt previously cast the deciding vote for the retro icons, so maybe let him weigh before making the final call.
Updated by Raymond Hoh over 3 years ago
If we want something a bit more neutral, but still random, identicon
is not bad.
Here's an example:
Updated by Sonja Leix over 3 years ago
That still looks retro to me :/
I'd opt for neutral, but am curious what Matt thinks.
Updated by Raymond Hoh over 3 years ago
Matt, do you want to chime in here about the site icon default and icon defaults in general?
For now, I'm going to go with what we had before just so we don't have a broken site icon display and can adjust again once we've made a decision. See https://github.com/cuny-academic-commons/cac/commit/7df419ef6d969cbb19478ed6d40657bd7ed99e07.
Updated by Boone Gorges over 3 years ago
- Target version changed from 1.18.5 to 1.18.6
Updated by Raymond Hoh over 3 years ago
From the dev call, the proposal is to change our default icons for sites and groups to use the BuddyPress defaults.
Site:
Group:
I also brought up whether we should use the BuddyPress default for users as well since we're making these icon changes. This is what the default user icon looks like:
Matt was kind of partial to identicons, but wasn't sure about whether the identicon color scheme would stand out enough.
We could decide to make these icon changes as part of the new redesign as well.
Updated by Sonja Leix over 3 years ago
Thanks Ray, I'm ok with whatever Matt wants to do, but might revisit this when we get to this part of the site during the redesign process.
Updated by Matt Gold over 3 years ago
Hi All -- thanks for giving me time to think about this. I'm fine with going with the gray icons if that's what the rest of the team prefers.
Updated by Boone Gorges over 3 years ago
- Target version changed from 1.18.6 to 1.18.7
Updated by Boone Gorges over 3 years ago
- Target version changed from 1.18.7 to 1.18.8
Updated by Boone Gorges over 3 years ago
- Target version changed from 1.18.8 to 1.18.9
Updated by Boone Gorges over 3 years ago
- Target version changed from 1.18.9 to 1.18.10
Updated by Boone Gorges over 3 years ago
- Target version changed from 1.18.10 to 1.18.11
Updated by Boone Gorges about 3 years ago
- Target version changed from 1.18.11 to 1.18.12
Updated by Boone Gorges about 3 years ago
- Target version changed from 1.18.12 to 1.18.13
Updated by Boone Gorges about 3 years ago
- Target version changed from 1.18.13 to 1.18.14
Updated by Boone Gorges about 3 years ago
- Target version changed from 1.18.14 to 1.18.15
Updated by Boone Gorges about 3 years ago
- Target version changed from 1.18.15 to 1.18.16
Updated by Boone Gorges about 3 years ago
- Target version changed from 1.18.16 to 1.18.17
Updated by Raymond Hoh about 3 years ago
- Related to Design/UX #14712: Default avatar images added
Updated by Raymond Hoh about 3 years ago
- Status changed from In Progress to Resolved
#14712 is a new ticket describing what we want to do for default avatars, so going to close this one.
Updated by Colin McDonald almost 3 years ago
Thanks for consolidating these, Ray.
Colin