Refined-github: Sticky file header covers permalinked line

[sompylasar]

Example

https://github.com/sindresorhus/refined-github/blob/7aad28997cdd9d6627c6620e242a17308f259897/source/content.js#L112

(navigate to this URL in a new tab)

Actual

Expected

[jgierer12]

Removed some comments for legibility.

Original discussion

bfred-it

Notice: this is not a bug of RGH

sompylasar

(@)bfred-it Well, yes, technically not a bug but a consequence of the fixed top bar feature.

bfred-it

No no, the top bar is fixed by GitHub. I didn't mean that as "won't fix this non-bug" but rather "we can improve this as an enhancement" 👍

sompylasar

Oops for some reason I thought it's refined, or without refinement it behaved differently, otherwise I would notice the inconvenience years ago.

bfred-it

It might be a new GitHub bug, can you report it to them? They might just fix it, I can see they're manually calling their function called scrollToTargetFragment so they are interested in getting it right.

sompylasar

(@)bfred-it Yes, will do.

:heart: bfred-it

bfred-it

My super-bad. It was indeed a bug of RGH. I disabled the extension and still saw it somehow.

[fregante]

I looked into fixing this via CSS and it doesn't seem to work with the padding hacks used for regular #hash linking, but because it's a table and it has that yellow background, some hacks are hard to apply. The pseudo-element hack also doesn't seem to work. I think this can only be fixed via JS.

Help wanted 😄

[fregante]

Note: this also affects markdown files: https://github.com/sindresorhus/refined-github/blob/master/readme.md#install (notice that covered Install heading)