Skip to content

Conversation

@kidonng
Copy link
Member

@kidonng kidonng commented Jul 28, 2020

Part of #3084 (if I should mention it)
Resolves #2091

Test: https://github.com/sindresorhus/refined-github/pull/636/files (unfold the diff above)

This PR simply warp the old code with selector-observer and add an additional selector (.blob-expanded) to make it work on unfolded diffs.

Merge format-conversation-titles in this PR as well (on request by #3397 (comment))

@fregante fregante marked this pull request as draft July 30, 2020 11:47
@fregante
Copy link
Member

#3410 (comment)

@fregante

This comment has been minimized.

@fregante
Copy link
Member

fregante commented Aug 8, 2020

I think this only needs to remove mentions of the issue title since that widget is incompatible with selector-observer

The rest seems correct

@fregante
Copy link
Member

fregante commented Aug 8, 2020

Also related: #2264 #3279

@kidonng kidonng marked this pull request as ready for review August 10, 2020 09:04
@kidonng
Copy link
Member Author

kidonng commented Aug 10, 2020

Also related: #2264 #3279

Roger that, but it would be in other PR

@fregante
Copy link
Member

Can you look into this as well? #3226 What's going on? Is the selector wrong?

@fregante fregante added the bug label Aug 10, 2020
@fregante fregante merged commit d01cff4 into refined-github:master Aug 10, 2020
@fregante
Copy link
Member

Congratulations for solving the oldest open bug 🍰

@fregante
Copy link
Member

Can you try restoring the laziness from https://github.com/sindresorhus/refined-github/pull/3091/files? I think my code only lacked the :not(.rgh-*) part of selectors. Otherwise this feature might be heavy on long files like https://github.com/yakov116/refined-github/blob/upstream/package-lock.json

@kidonng
Copy link
Member Author

kidonng commented Aug 28, 2020

Is the selector wrong

Yeah, need to narrow it down

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

Labels

Development

Successfully merging this pull request may close these issues.

Apply auto-linking after unfolding diffs

2 participants