repo-avatars - Fix wrong organization avatar

[LitoMore]

Closes #7322

I've added a new optional parameter ignoreBots, for the repo-avatars scene.

Test URLs

• Fix typos simple-icons/simple-icons#10728

[fregante]

Thank you for the PR! Sorry for the delay, I wanted to think about this for a bit.

I think that rather than a flag like this, it should just change the way it selects existing avatars in every case.

There could be a @LitoMore user and a "LitoMore" app. The alt text provided by GitHub is actually incorrect, you cannot @-mention apps, you can only @-mention actual user/orgs (sometimes this overlaps, sometimes this doesn't)

So instead of querying img[alt="@simple-icon"] it should query [href="/simple-icon" i] img. I think that's what we had at some point, so we should git blame the line and see why it was changed.

[LitoMore]

We should git blame the line and see why it was changed.

Should be this: #2142

[fregante]

Indeed! So that selector is new specifically to add support for bots, the /user was just a raw request URL.

My suggested selector should work

[fregante]

Thank you! Have you tested this change? Does it fix the issue for you?

[LitoMore]

@fregante I've tested it on my side. It's working correctly.