Bug 429636 - When i click brush editor second time, it stops/freezes the app.
Summary: When i click brush editor second time, it stops/freezes the app.
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Usability (other bugs)
Version First Reported In: 4.4.1
Platform: Android Android 7.x
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2020-11-25 12:22 UTC by tolgayamannnn
Modified: 2021-01-06 13:19 UTC (History)
1 user (show)

See Also:
Latest Commit: https://invent.kde.org/graphics/krita/commit/afa491eb33b1fcbef3d022386f5b9369f2835643
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
System info (2.31 KB, text/plain)
2020-11-25 12:22 UTC, tolgayamannnn 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description tolgayamannnn 2020-11-25 12:22:26 UTC 
Created attachment 133640 [details]
System info

When i click brush editor twice,i mean clicking it again. It stops the app,can't do anything.Freezes i
Comment 1 Halla Rempt 2020-11-25 12:38:29 UTC 
I'm sorry, I cannot reproduce this either on Android 10 or on ChromeOS. I don't have access to an older version of Android.
Comment 2 Bug Janitor Service 2021-01-06 13:17:03 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/graphics/krita/-/merge_requests/639
Comment 3 sh_zam 2021-01-06 13:19:00 UTC 
Git commit afa491eb33b1fcbef3d022386f5b9369f2835643 by Sharaf Zaman.
Committed on 06/01/2021 at 13:18.
Pushed by szaman into branch 'master'.

Android: fix app freeze problem caused by pop-up widgets

Reason: If we double press KisPopupButton, for our first press we tell
Qt to widget->show() which casues an internal flush of event queue then
the second event is sent down which hides the widget.

The second event is immature because the actions of first event didn't
complete in time. This makes the widget hidden while also grabbing mouse
and keyboard, thus freezing the entire app.

Note: The bug is a very time sensitive.

A  +28   -0    3rdparty/ext_qt/0055-Android-prevent-out-of-order-events.patch
M  +1    -0    3rdparty/ext_qt/CMakeLists.txt

https://invent.kde.org/graphics/krita/commit/afa491eb33b1fcbef3d022386f5b9369f2835643