Popup window is broken

[morawcik]

Hi Starts from couple days ago most time when I'm trying open popup window it opens normal but then very fast switch to so small like 50x50px. There are a times when it opens normal and stay normal but it's like 20% from every opening.

[olsh]

Hi, could you please specify your browser?

[morawcik]

Google Chrome was 65.* (not know details) and now 66.0.3359.139 (64 bits) - broken on both

[navarr]

I am also having this issue on Chrome 66.0.3359.139 (Mac OS X High Sierra).

Occasionally it works fine, but most of the time it shrinks back to the small window.

[olsh]

I can't reproduce this issue on Windows 10. @morawcik are you using Mac too?

[morawcik]

Yes, Mac OS X Sierra and High Sierra

[olsh]

Probably related to https://github.com/olsh/Feedly-Notifier/blob/master/src/scripts/popup.js#L190-L193

I'll try to find a Mac to test.

[olsh]

I can't reproduce this issue on a test Mac. So it would be nice if anybody helps me with this issue.

I'm 99% sure that this issue is caused this bug: https://bugs.chromium.org/p/chromium/issues/detail?id=307912 I tried workaround this bug with this code but it seems like it's not a perfect solution.

[morawcik]

What help?

I think that it can be this bug but I had this problem only with Feedly Notifier.

[madhur]

I am having same issue.

Running MAC OSX Sierra 10.12.6

Chrome Version 66.0.3359.139 (Official Build) (64-bit)

[olsh]

Guys, if you can, please build the extension from the sources and play with the code (remove it, increase interval etc.): https://github.com/olsh/Feedly-Notifier/blob/master/src/scripts/popup.js#L190-L193

Here are the build steps: https://github.com/olsh/Feedly-Notifier#build

[morawcik]

I removed this if for mac and popup with login button works fine but trying login I'm getting "invalid redirect_uri" error

[olsh]

Are you using the lastes keys for the sandbox? https://groups.google.com/forum/#!topic/feedly-cloud/lvGXcO5uzas

[morawcik]

Yes. I need do/change sth more than this kay for command line?

[akaSybe]

I'll take a look in the evening

[morawcik]

But @olsh before removed this if statement even popup with login button was broken so maybe this is solution for all this problem?

[olsh]

@morawcik just pushed a fix for the sandbox build, please rebuild the extension.

But @olsh before removed this if statement even popup with login button was broken so maybe this is solution for all this problem?

Probably, but we need to test this. Here is the long thread about the issue: https://github.com/olsh/Feedly-Notifier/issues/59

[morawcik]

Thanks. So removing if statement helps but it's not a solution - popup still resizing to small version but much less often.

[morawcik]

I changed timeout to 500 and no issue for couple of min opening popup. I'll write if I'll get it again.

[olsh]

@morawcik how the testing is going, any issues?

Could you please send a PR if everything is ok?

[akaSybe]

I increased timeout for Mac. New version will be available in store soon.

[morawcik]

Nice, thanks @akaSybe

Sorry @olsh I just back to home. It works great - no single problem :)

[olsh]

Cool, I'm closing this issue out then. Feel free to reopen it if you still have the bug.