Open xt0rted opened 5 years ago
Does this still need to be addressed? Right now, everything looks okay to me.
I'm not sure, I never actually used this style. From what I recall the changes that needed to be made were around removing a few of the selectors using :not()
since this is built-in now and duplicate rules were being added from it.
https://github.blog/changelog/2019-03-18-sticky-file-headers-in-pull-requests/
Looks like they're using the class
.sticky-file-header
to mark these up so you'll need to adjust some of the selectors to be.file-header:not(.sticky-file-header)
.