Skip to content

Conversation

@FloEdelmann
Copy link
Member

@FloEdelmann FloEdelmann commented Jun 12, 2020

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.

Unbenannt

.replace(/^an?/, '1')
.split(' ');

const secondSidebarSection = await elementReady('.repository-content .BorderGrid-row + .BorderGrid-row');
Copy link
Member

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.

Suggested change
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');

FloEdelmann and others added 2 commits June 12, 2020 20:43
Copy link
Member

@fregante fregante left a 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants