Position: sticky polyfill (follow-up from "Upgrade to Bootstrap 4")
The following discussion from !18232 (merged) should be addressed:
-
@filipa started a discussion: (+5 comments) there are more places that need this. what is our approach about the polyfils? Would it make more sense to load it in the main bundle?
via @iamphill
CSS.supports('position: sticky')
will returntrue
orfalse
if it is supported or not. Some newer browser may not support sticky but supportCSS.supports
. IE doesn't supportCSS.supports
& doesn't supportsticky
so we can fallback to an affix plugin.Either way, i'd prefer all of this to ideally go through the sticky util (https://gitlab.com/gitlab-org/gitlab-ce/blob/master/app/assets/javascripts/lib/utils/sticky.js) if possible. May be too big for this merge request, so may need a tech debt follow-up.
/cc @ClemMakesApps