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

file-age-color also runs on issues and PR tab #6366

Closed
Katsute opened this issue Feb 20, 2023 · 5 comments · Fixed by #6436
Closed

file-age-color also runs on issues and PR tab #6366

Katsute opened this issue Feb 20, 2023 · 5 comments · Fixed by #6436
Labels
bug help wanted Please! ♥︎ Particularly useful features that everyone would love!

Comments

@Katsute
Copy link
Contributor

Katsute commented Feb 20, 2023

Description

The new file age color feature also changes colors on issue and PR tabs.

files

How to replicate the issue + URL

  1. This repo https://github.com/refined-github/refined-github/
  2. Press issue or pr tab

Only happens if you go directly from a file page to a issue/pr tab, moving from issue to pr tab doesn't have this issue.

Extension version

23.2.20

Browser(s) used

Google Chrome 110.0.5481.104 x64

@Katsute Katsute added the bug label Feb 20, 2023
@fregante fregante added help wanted disabled via hotfix A label to remind us to remove the feature from yolo labels Feb 20, 2023
fregante added a commit to refined-github/yolo that referenced this issue Feb 20, 2023
@fregante
Copy link
Member

How could that be possible? We're using the correct page filters and signal 🤔

function init(signal: AbortSignal): void {
observe([
'.js-navigation-item relative-time', // TODO: Drop old view in mid 2023
'.react-directory-commit-age > [title]',
], addHeatIndex, {signal});
}
void features.add(import.meta.url, {
include: [
pageDetect.isRepoTree,
],
init,
});

It feels like I broke something important or it has always been broken and this feature just made it visible

@Katsute
Copy link
Contributor Author

Katsute commented Feb 20, 2023

Might be a timing issue caused by the SPA. Is it possible the feature is running after the DOM change but before the URL change? This might explain why going to another tab right after doesn't have this issue.

@fregante
Copy link
Member

timing issue caused by the SPA

Last related change in 23.2.1:

@Shinigami92
Copy link
Contributor

Oh, I thought that was a feature 👀
I just wanted to report the bug that it doesn't load when you directly go to issue or PRs e.g. by reloading the page
You first need to load the Code tab and then navigate to issue/PR

@fregante fregante added the Please! ♥︎ Particularly useful features that everyone would love! label Mar 2, 2023
@5HT2
Copy link

5HT2 commented Mar 16, 2023

I also honestly thought this was an intentional feature. If there could be an option to also enable this for issues / pull requests I'd greatly appreciate that!

@fregante fregante removed the disabled via hotfix A label to remind us to remove the feature from yolo label May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help wanted Please! ♥︎ Particularly useful features that everyone would love!
Development

Successfully merging a pull request may close this issue.

4 participants