Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong highest-rated-comment if there's >1 reactions #7687

Closed
1 task done
TonyRL opened this issue Aug 14, 2024 · 8 comments
Closed
1 task done

Wrong highest-rated-comment if there's >1 reactions #7687

TonyRL opened this issue Aug 14, 2024 · 8 comments
Labels
bug firefox Related to Firefox only help wanted

Comments

@TonyRL
Copy link

TonyRL commented Aug 14, 2024

Description

The highest rated comment should be the one from Tennyleaz (42 upvotes) instead of the one from xromansx (12 upvotes).

Screenshot

Screenshot

Screenshot 2

Screenshot 2

How to replicate the issue + URL

Visit: netbirdio/netbird#363

Extension version

24.8.13

Browser(s) used

FireFox v129.0.1

Token ⚠️

  • I set a token in the options page. I understand that some features require a token to work.
@TonyRL TonyRL added the bug label Aug 14, 2024
@fregante
Copy link
Member

fregante commented Aug 15, 2024

It picks the 42 upvotes for me (Safari, v24.7.20)

image

@TonyRL
Copy link
Author

TonyRL commented Aug 15, 2024

It works fine on Chromium-based browsers (v118 and v127). So I guess it's a Firefox specific issue.

@fregante fregante added firefox Related to Firefox only help wanted labels Aug 15, 2024
@fregante
Copy link
Member

Does it happen consistently? Or just sometimes?

@TonyRL
Copy link
Author

TonyRL commented Aug 15, 2024

/*
Test URLs:
- 8th comment, has link: https://github.com/refined-github/refined-github/issues/4166
- 2nd comment, no link: https://github.com/refined-github/refined-github/issues/825
*/
Works as intended on Firefox.

I haven't found any GitHub issues that can reproduce the same result yet. I've tried disabling all extensions except Refined GitHub and it's still picking up the 12 upvotes one.
However, it's picking up the correct one in incognito mode.

@fregante
Copy link
Member

I can't repro in Firefox either, Refined GitHub v24.8.13 and the previous v24.8.3

Screenshot 8

The feature waits for the whole page to be loaded too, so this shouldn't be a race condition either. Do you have other extensions?

Does it happen at every load of netbirdio/netbird#363 or just sometimes?

@fregante
Copy link
Member

Could something like this have been the problem? Are you logged into GitHub when the issue occurs?

GitHub sometimes trims the reactions and I don’t think we support this DOM

IMG_4703

@TonyRL
Copy link
Author

TonyRL commented Aug 18, 2024

Do you have other extensions?

...I've tried disabling all extensions except Refined GitHub and it's still picking up the 12 upvotes one...

Originally posted by TonyRL in #7687 (comment)

Are you logged into GitHub when the issue occurs?

Yes. It's logged in on Firefox. If I log out (or in incognito mode) when visiting that issue, it does pick up the 42 upvotes one.

Does it happen at every load of netbirdio/netbird#363 or just sometimes?

It happens on every load.

@fregante
Copy link
Member

fregante commented Nov 8, 2024

Thanks for the report, but seems to be too niche to work on. Likely the new view will fix it or require a rework of the feature anyway.

@fregante fregante closed this as not planned Won't fix, can't repro, duplicate, stale Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug firefox Related to Firefox only help wanted
Development

No branches or pull requests

2 participants