-
Notifications
You must be signed in to change notification settings - Fork 131
Header (and all fixed elements) only stays fixed on mobile #42
Comments
Hi @TheDolo, hm, I didn't consider it like an issue, but you're right this needs some research for users that are interested in fixed Header. I think it's an "enhancement", not a "bug". Will try to figure out what's the problem. |
It's definitely a problem if the only fixed element allowed on the page is the sidebar! Edit: if the page is too small for a scroll (no overflow), fixed elements work fine. So I'll insert a react custom scroll bar into a max height page for . now |
Oh, I just misunderstood your sentence, ok, I'll investigate this case with fixed elements on the page (desktop), haven't seen this problem before. |
This is the culprit in semantic-ui-css module node_modules/semantic-ui-css/semantic.css starting around line:34342 /* Left Visible */
.ui.visible.thin.left.sidebar ~ .fixed,
.ui.visible.thin.left.sidebar ~ .pusher {
-webkit-transform: translate3d(150px, 0, 0);
transform: translate3d(150px, 0, 0);
}
I'll look into a fix. |
@TheDolo you can use |
Sorry, people. Not quite had the time yet. I suppose this is just a starter, really. As Metnew said - it'll work outside the main container but some styling tweaks with margins and padding will be required. |
Page structure was improved + react-headroom was added. Now anything can be fixed/statis/relative/etc. Anything supported. |
@Metnew it still doesn't work. Header (react-headroom) scrolls on desktop. |
Hm, ok, I'll take a look asap |
Not sure, but the possible root of the problem:
I had a project where this feature works as expected. I'll take a look more detailed later |
I'm submitting a ...
Bug Report:
I know this is probably part of the Pusher issues.
Currently, fixed elements on desktop will not stay fixed and will scroll. Once mobile (menu is gone), all issues are resolved.
The text was updated successfully, but these errors were encountered: