- Issue created by @fjgarlin
- π§πͺBelgium BramDriesen Belgium π§πͺ
I think it should be a combination of both when this happens?
Meaning:
- Remove duplicates
- Show a list of companies below the badge "Associated with organisation A, B, C"But I would only show the "Associated with" message when there are duplicates. Because in all other cases it's quite clear to which org the badge belongs.
- πͺπΈSpain fjgarlin
Because in all other cases it's quite clear to which org the badge belongs.
If a badge only belongs to one company, you wouldn't know which one it is unless you visit the company profile tho.
- πͺπΈSpain pcambra Asturies
I'd be one of the users """affected""" by this, not sure if this change is worth implementing, however I would be more in favor of a group by company/organization rather than removing the duplicates. I do like the "Associated with" naming suggested by @BramDriesen
- πͺπΈSpain fjgarlin
With that approach, if we show all the text it can get out of hand easily...
Maybe show the tooltip on hover?
- π§πͺBelgium BramDriesen Belgium π§πͺ
Not sure #5 is that bad actually. Looks a lot less cluttered as showing duplicate badges.
- πΊπΈUnited States drumm NY, US
A potential solution is putting the badges right next to the organization logo on the left, matching the badge size to the logo size. That makes it more clear where the badges come from. If an organization has more than one badge, then it would have to go to the next line, which would probably be okay in most cases. I do like that it frees up some page real-estate to focus more on the person, but does split up the supporting badges.
If we stick with the right side, letβs make the text a bit more descriptive, like:
- Cambrico, Ymbra are Bronze Drupal Certified Partners
- Tag1 is a Platinum Drupal Certified Partner
That helps if you get lost deciding if the text goes with the badge above or below, and is a bit more direct than βAssociated withβ
- First commit to issue fork.
- Merge request !282Stop rendering badges as we discover them, instead build a list of badges and... β (Merged) created by B_man
- Status changed to Fixed
3 months ago 10:29pm 9 September 2024 Automatically closed - issue fixed for 2 weeks with no activity.