I clicked on “Feed Types” on the demo, and was happy that once I select an article, I can use j/k to move between articles. However, if you are on an article and scroll down (e.g. with spacebar), then hit j or k, you go to the next article scrolled down and have to scroll back up to the top. This is a bit awkward. If you go to an article, either it should always go to the top, or it should remember where you were for that particular post—not whatever you were previously looking at.
Thanks for the bug report! This seems to be a demo-only bug (the demo has some custom state logic since it’s all hard-coded instead of doing real queries), but I’ll look into why it’s inconsistent with the real app.
Update: The bug was in the small amount of state code that’s impractical to share with the main app (for now), but it’s fixed in this PR and should work correctly in the demo now.
This is pretty cool!
One tiny feature request / bug report.
I clicked on “Feed Types” on the demo, and was happy that once I select an article, I can use j/k to move between articles. However, if you are on an article and scroll down (e.g. with spacebar), then hit j or k, you go to the next article scrolled down and have to scroll back up to the top. This is a bit awkward. If you go to an article, either it should always go to the top, or it should remember where you were for that particular post—not whatever you were previously looking at.
Thanks for the bug report! This seems to be a demo-only bug (the demo has some custom state logic since it’s all hard-coded instead of doing real queries), but I’ll look into why it’s inconsistent with the real app.
Update: The bug was in the small amount of state code that’s impractical to share with the main app (for now), but it’s fixed in this PR and should work correctly in the demo now.