Conversation
_includes/inline-style.css
Outdated
There was a problem hiding this comment.
I'd prefer auto, scroll will always render the scroll bar even if it's not needed
_layouts/home.html
Outdated
There was a problem hiding this comment.
style="" really shouldn't be used, put it in the appropriate <style> tag
_includes/inline-style.css
Outdated
There was a problem hiding this comment.
using position: sticky without an inset (top/bottom/left/right) feels odd, it's got no place to stick to. what's the desired effect?
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
_includes/inline-style.css
Outdated
There was a problem hiding this comment.
this feels like a bad idea, why hide it?
_includes/inline-style.css
Outdated
There was a problem hiding this comment.
feels like this will have the effect that the footer will always be on screen? instead of being off screen on long pages?
benharri
force-pushed
the
aside-scroll-independently
branch
from
77223bd to
95804f8
Compare
swantzter
added
Area: General
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
4 participants
this feel kinda janky, let me know if you can think of a better way to do this
closes #39