Changes in Swage theme for Mark As Read on Scroll#2094
Conversation
Remove theme sizing to make scroll as read work per FreshRSS#1980
|
Thanks. Alternative suggestions welcome. |
|
A simple decoupling could be done by just removing sticky-kit, and would leave the feed list at the top, like the behavior of feeds in versions prior to... 1.5, it looks like (#1214)? Without sticky-kit, individual scrolling of the sidebar and main feeds could be accomplished purely through css. I've tossed together a quick proof-of-concept here: https://github.com/pattems/FreshRSS/tree/Sticky_aside This isn't a terribly robust solution, however. There's problems with the last feed or two being obscured by nav buttons if they're visible, but with a bit more work it's a viable, if slightly hacky, pure-css solution. I'm not terribly good with Javascript, but I think I can probably throw together a JS replacement for sticky-kit that will let us have those decoupled feeds, it might just need a little extra polishing by more capable hands after I'm done with it. I'm not making any promises, but I hope to have something together by My Friday evening/Your Saturday morning. |
Remove theme sizing to make scroll as read work per FreshRSS#1980
Remove theme sizing to make scroll as read work per #1980, as requested in #2088 (comment)
I just want to say that I really don't like this solution; it doesn't seem intuitive to me if scroll to mark as read is enabled, and looks downright broken if you don't, which is why I'd overridden the height in the first place. I'll be adding looking into another way to do this to my personal to-do list.