search

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
267 stars 38 forks source link

Popup body/content max-height too low when page zoom is in use #179

Closed pullasuti closed 3 years ago

pullasuti commented 3 years ago

If default page zoom in Chrome settings is set to over 100%, the content of the notifier overflows and gets two scrollbars.

Steps to reproduce the behavior:

  1. Go to Chrome -> Settings -> Appearance
  2. Select 'Page zoom' higher than 100%
  3. Set max number of feeds to more than 6 in Feedly Notifier
  4. Open Feedly notifier popup
  5. See too many scrollbars

Expected behavior No double scrollbars. The static max-height in styles for popup-body and popup-content should be configurable/more responsive. 600px causes issue.

Screenshots image

Desktop (please complete the following information):

olsh commented 3 years ago

Hi guys, I saw the PR. Sadly I can't test it right now because I can't reproduce the issue on my laptop (macOS). I'll try to test it soon when I get my Windows PC.