`linkify-line-numbers` - Fix occasional layout break by SunsetTechuila · Pull Request #9532 · refined-github/refined-github · GitHub
Skip to content
1 change: 1 addition & 0 deletions build/__snapshots__/features-meta.json
7 changes: 7 additions & 0 deletions source/features/github-bugs.css
Original file line number Diff line number Diff line change
Expand Up @@ -303,3 +303,10 @@ review-thread-collapsible > .js-toggle-outdated-comments {
.ActionListItem-singleSelectCheckmark {
transition: none !important;
}

/* Prevent empty diff cells from looking clickable */
/* Info: https://github.com/refined-github/refined-github/pull/9532#pullrequestreview-4321433264 */
/* Test: https://github.com/refined-github/sandbox/pull/81 */
.empty-cell {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Element with the empty-cell class can also have the blob-num class, which sets the cursor to pointer. Having cursor: pointer on a non-interactive element doesn't make sense.

cursor: default !important;
}
6 changes: 6 additions & 0 deletions source/features/linkify-line-numbers.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
html:not([rgh-OFF-linkify-line-numbers]) {
.blob-num:not(.blob-num-hunk, .empty-cell):hover {
cursor: pointer;
text-decoration: underline;
}
}
29 changes: 21 additions & 8 deletions source/features/linkify-line-numbers.tsx
Loading