Skip to content

Comments

Add linkify-user-labels feature#5765

Merged
fregante merged 5 commits intomainfrom
linkify-user-labels
Jul 4, 2022
Merged

Add linkify-user-labels feature#5765
fregante merged 5 commits intomainfrom
linkify-user-labels

Conversation

@fregante
Copy link
Member

@fregante fregante commented Jul 3, 2022

Screenshot

Screen Shot

Test URLs

Bot PR
webpack/webpack#15926 (comment)

Bot comment
webpack/webpack#15926 (comment)

Bot commented on behalf of
webpack/webpack#15926 (comment)

Member review
#5721 (review)

Contributor review comment
#5691 (comment)

Contributor review second comment
#5691 (comment)

Contributor review second comment in Files tab
https://github.com/refined-github/refined-github/pull/2667/files#r366433031

@fregante fregante marked this pull request as ready for review July 3, 2022 09:22
@returns dependabot[bot]

*/
export default function getCommentAuthor(anyElementInsideComment: Element): string {
Copy link
Member

@yakov116 yakov116 Jul 3, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we reuse this helper for show-names?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was the idea, just not as part of this PR

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 great minds think alike 😜

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added to #5222

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

show-names has to query the exact author link so this function is not useful. I don't see any other situations where it can be used (grepped author)

@fregante fregante merged commit 327747a into main Jul 4, 2022
@fregante fregante deleted the linkify-user-labels branch July 4, 2022 00:55
@kidonng
Copy link
Member

kidonng commented Jul 6, 2022

Side effect on review comments:

image

Normal:

image

To be honest I kinda like it somehow emphasis the author. And it's GitHub being inconsistent here so...

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

Development

Successfully merging this pull request may close these issues.

Link "Contributor" label to commit list

4 participants