Skip to content

small-user-avatars - Support repo avatars on notifications page#9472

Merged
fregante merged 8 commits into
mainfrom
avatars-repo-list
May 24, 2026
Merged

small-user-avatars - Support repo avatars on notifications page#9472
fregante merged 8 commits into
mainfrom
avatars-repo-list

Conversation

@SunsetTechuila

@SunsetTechuila SunsetTechuila commented May 15, 2026

Copy link
Copy Markdown
Contributor

Resolves #9468

Test URLs

https://github.com/notifications

Screenshot

image

@fregante

Copy link
Copy Markdown
Member

It would be great to not implement every feature that is requested and not discussed/approved over something that has been "help wanted" for a while

@SunsetTechuila

Copy link
Copy Markdown
Contributor Author

I liked that suggestion. You're free to close this PRs if you don't

@fregante

Copy link
Copy Markdown
Member

I'm ok with it too, it's just a general advice, which is also why I didn't react or immediately accept the issue

@SunsetTechuila SunsetTechuila marked this pull request as ready for review May 23, 2026 14:04
just to keep changes minimal. It's not required
@fregante

Copy link
Copy Markdown
Member

The concern here is that there isn't enough space, the repos already wrap so easily:

Screenshot 21

GitHub has a similar UI with the avatars on the dashboard, but in the same viewport size they have more breathing space:

Screenshot 22

I think at some point someone somewhere asked to drop the org/user name in this list specifically to make more room. I think that request makes more sense.

@SunsetTechuila

Copy link
Copy Markdown
Contributor Author

Fair. Addressed this

@fregante fregante merged commit c70caed into main May 24, 2026
13 checks passed
@SunsetTechuila SunsetTechuila deleted the avatars-repo-list branch May 24, 2026 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Show repo owner's profile picture in Notifications page's left repositories sidebar

2 participants