olsh / Feedly-Notifier

Google Chrome, Firefox, Opera and Microsoft Edge extension for reading news from RSS aggregator Feedly

http://olsh.github.io/Feedly-Notifier/

Mozilla Public License 2.0

273 stars 38 forks source link

Feed `overflow-y: scroll` to avoid buttons move on read item #199

Closed mathiash98 closed 2 years ago

mathiash98 commented 2 years ago

Problem:

More than x items in feed results in scrollbar -> Removing items by cliking the mark as seen buttons leads to item removed -> Which then results in "mark as seen"-button moving physically on screen -> Which then breaks the flow of filtering news

Video:

https://user-images.githubusercontent.com/5204006/147878336-60a8a52a-b7a5-466f-93e7-f15402533ca0.mp4

Drawbacks of this solution:

Scrollbar is always visible, could be annoying for people that never have more than x items and uses the "mark as seen"-button.

Please verify if overflow-y: scroll should only be on #feed or on #feed, #feed-saved

olsh commented 2 years ago

Hi @mathiash98

Sorry for the super-late response. Sadly, I can't reproduce the issue, could you please specify your browser and plugin version?

mathiash98 commented 2 years ago

Hi @mathiash98

Sorry for the super-late response. Sadly, I can't reproduce the issue, could you please specify your browser and plugin version?

No problem. I am using chromium based browsers, so both edge og laptop and chrome on desktop. Feedly version 2.26.2.

Do you have enough items in the list such that the scrollbar appears? The "issue" is that all the buttons move some pixels to the right when scrollbar dissappears due to few items in list, such that I need to move the mouse to proceed.

olsh commented 2 years ago

Do you have enough items in the list such that the scrollbar appears?

Yes, in my case the scroll disappears correctly. Does the issue happen randomly? Could you please share your RSS feed? Maybe there is something with it?

I've noticed that on your video the are duplicate RSS items.

olsh commented 2 years ago

Ohh, I see, I thought that was the problem with invalid feeds rendering. Well, I prefer to hide the scroll. I think the proper solution is to use custom scrollbars.

mathiash98 commented 2 years ago

Ohh, I see, I thought that was the problem with invalid feeds rendering. Well, I prefer to hide the scroll. I think the proper solution is to use custom scrollbars.

Yeah I see why it was misunderstood.

I won't disagree with your reasoning where you prefere the scrollbar goes away. Need a better solution then for my desired behavior