-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Restore repo-age in "Repository refresh" beta
#3225
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
Conversation
| .replace(/^an?/, '1') | ||
| .split(' '); | ||
|
|
||
| const secondSidebarSection = await elementReady('.repository-content .BorderGrid-row + .BorderGrid-row'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should start marking these selectors so they can be all found and replaced once the issue is fixed.
| const secondSidebarSection = await elementReady('.repository-content .BorderGrid-row + .BorderGrid-row'); | |
| // TODO: simplify selector after https://github.com/sindresorhus/element-ready/issues/29 | |
| const secondSidebarSection = await elementReady('.repository-content .BorderGrid-row + .BorderGrid-row'); |
Co-authored-by: Fregante <opensource@bfred.it>
fregante
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works, but it slows down the non-beta version since now elementReady will wait until the whole page has loaded.
References #3081, follows #3201 (comment).
Test this on any repo, e.g. https://github.com/sindresorhus/refined-github. Note that you should clear the cache to test if the commit link can be extracted correctly.