Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upload animation freezes #1503

Open
mmahmoudian opened this issue Mar 22, 2021 · 15 comments
Open

Upload animation freezes #1503

mmahmoudian opened this issue Mar 22, 2021 · 15 comments
Assignees
Labels
Bug It's a bug Visuals Graphical-related

Comments

@mmahmoudian
Copy link
Member

mmahmoudian commented Mar 22, 2021

Flameshot version
Flameshot v0.9.0
Compiled with Qt 5.15.2

Describe the bug
The upload animation freezes and this causes the users to panic (#1501). Also, to my surprise (I repeated this 3 times and got the same exact result), the clock on the screen also stops working for half of the duration of uploading process!

I suggest the Latest Uploads window opens after the upload is completed.

Peek 2021-03-22 15-20

To Reproduce

  1. flameshot gui
  2. select the region
  3. click on upload to Imgur
  4. the animation freezes and users panic

Expected behavior

Smooth animation.

System Information

System:
    Host: hostname Kernel: 5.9.16-1-MANJARO x86_64 bits: 64
    Desktop: KDE Plasma 5.21.3 Distro: Manjaro Linux
Graphics:
    Device-1: Intel driver: i915 v: kernel
    Device-2: AMD Lexa XT [Radeon PRO WX 3200] driver: amdgpu v: kernel
    Display: x11 server: X.Org 1.20.10 driver: loaded: amdgpu,ati,intel
    unloaded: modesetting resolution: 1: 1920x1080~60Hz 2: 1080x1920~60Hz
    OpenGL: renderer: AMD Radeon Pro WX 3200 Series (POLARIS12 DRM 3.39.0
    5.9.16-1-MANJARO LLVM 11.1.0)
    v: 4.6 Mesa 20.3.4
@mmahmoudian mmahmoudian added Bug It's a bug Visuals Graphical-related labels Mar 22, 2021
@borgmanJeremy
Copy link
Contributor

@mmahmoudian this should be fixed on the master branch. Can you test that? I really need to get a point release out.

@mmahmoudian
Copy link
Member Author

I tried it with the AppImage of #1526 and got this:

image

I then tried with AppImage of #1524 (which is the master) and got the same exact error. I wonder if it has something to do with AppImage. Should I try to compile it locally?

@borgmanJeremy
Copy link
Contributor

Yeah either build from source or do a cleanbuild from AUR. Looks like the app image is missing the SSL binaries

@giswqs
Copy link

giswqs commented Apr 4, 2021

I am having the same issue. Flameshot freezes for about 10 seconds when uploading GIF to imgur.com

image

@mmahmoudian mmahmoudian self-assigned this Apr 5, 2021
@coozoo
Copy link

coozoo commented May 6, 2021

actually for me the same freeze happens every time when I'm trying to copy into clipboard, just in difference that all screen frozen
the only way for me to switch tty and kill flameshot
but in console it's the only stuff that I see

$ flameshot
QSettings::value: Empty key passed
QSettings::value: Empty key passed
QSettings::setValue: Empty key passed
QSettings::value: Empty key passed
QSettings::setValue: Empty key passed
QPainter::begin: Paint device returned engine == 0, type: 2
QPainter::setRenderHint: Painter must be active to set rendering hints
QPainter::setCompositionMode: Painter not active
QPainter::translate: Painter not active
QPainter::setPen: Painter not active
QPainter::setBrush: Painter not active
QPainter::setBrush: Painter not active
Terminated

Sure terminate it's only after killing by hands
mine version flameshot-0.9.0-1.fc33.x86_64

@mmahmoudian
Copy link
Member Author

@coozoo How many seconds should you wait until it unfreezes itself without you killing it?

@coozoo
Copy link

coozoo commented May 6, 2021

@coozoo How many seconds should you wait until it unfreezes itself without you killing it?

Oh about 15 seconds and I thought that's all and nothing will happen.

So I've tried to wait more, few times. First one unfreeze after ~30 seconds. 2nd - after about ~45, one more time and it's about 50s 😕

@dausruddin
Copy link

Can confirm. Installed flameshot and having the same issue as OP.
Uploaded files exist as suggested in #1501, but I wish there will be no freeze at all since it also freezes some parts of my desktop.

@mmahmoudian
Copy link
Member Author

I just checked with the latest commit in master Flameshot v0.10.1 (203b5ba) and it seems this is resolved. At least it doers not happen anymore.

@dausruddin would you please check this as well

@mmahmoudian mmahmoudian removed their assignment Dec 8, 2021
@dausruddin
Copy link

Sorry, a bit late. But I can confirm the issue is no more.

Thanks.

@OnkelTem
Copy link

OnkelTem commented Jan 29, 2022

Still persists on:

Flameshot v11.0.0 ()
Compiled with Qt 5.12.8
linux: 5.13.0-27-generic
ubuntu*: 20.04

Kubuntu actually.

Thanks to this comment I discovered that uploads are actually successful:
#1501 (comment)

@mmahmoudian
Copy link
Member Author

@OnkelTem how have you installed Flameshot?

@mmahmoudian
Copy link
Member Author

mmahmoudian commented Jan 30, 2022

@borgmanJeremy I can confirm that it is again happening. @OnkelTem Thanks for reporting. I'm re-opening this.

I also realized that when the animation is frozen, whatever change the user make to the window dimension (because the circle is not fully fitted in the window), the whole window geometry glitches (shows the desktop wallpaper)

Flameshot v11.0.0 (1cc5a26)
Compiled with Qt 5.15.2
linux: 5.15.16-1-MANJARO
manjaro: unknown

@mmahmoudian mmahmoudian reopened this Jan 30, 2022
@borgmanJeremy
Copy link
Contributor

Disable "Copy URL after upload" for now. The bug is related to a race condition between the clipboard and the system notification....again

@borgmanJeremy
Copy link
Contributor

#2348 seems to fix it for me. Will merge after the CI finishes.

@borgmanJeremy borgmanJeremy self-assigned this Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug It's a bug Visuals Graphical-related
Projects
None yet
Development

No branches or pull requests

6 participants