Problem/Motivation

It would be good to move layout support for sticky navbar and absolute header to bs_base and generalize&clean SASS/CSS rules.

Related to #3437070: Improve sticky navigation behavior which is probably a follow-up or can be merged with this issue.

Comments

pivica created an issue. See original summary.

ytsurk’s picture

ytsurk
Location Switzerland
commented

Make sure to use translate3d(0,0,0) on the fixed positioned element to kick in hardware-acceleration :D
This then also wants the sticky's z-index (and thus position: relative) on the non-sticky navbar-wrapper, otherwise an expanded menu can look broken/glassy/half-transparent !

Or maybe even use position: sticky - which won't work with the page_top region ...,
and is quite fragile. (https://stackoverflow.com/questions/43707076/position-sticky-not-working...)

pivica’s picture

pivica commented

The #3033775: Sticky header browser compability issue description and comments are related to this.

pivica’s picture

pivica commented
Version: 8.x-1.x-dev » 2.x-dev

Thinking about this seems to me it is better to push this to 2.x version. It is not critical for 1.x and also we need to port this from custom internal theme which means config update problems, etc. Better to do this for next major version release.

pivica’s picture

pivica commented
Issue summary: View changes