Bug 462848 - [Android] Touch Docker's File buttons open a white screen and lock up
Summary: [Android] Touch Docker's File buttons open a white screen and lock up
Status: REPORTED
Alias: None
Product: krita
Classification: Applications
Component: Dockers (show other bugs)
Version: nightly build (please specify the git hash!)
Platform: Android Android 13.x
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-12-10 13:37 UTC by Tyson Tan
Modified: 2022-12-24 09:37 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tyson Tan 2022-12-10 13:37:33 UTC
On Androids tablets, the File buttons (Open, Save, Save as) in Touch Docker are broken. They can only bring up a full screen white window. Press ESC on the keyboard restores a title bar. But nothing is happening, and nothing can be done. There is no way to exit that state, either.

On the other hand, the File menu, File buttons in the toolbars were all working as intended.

Tested with:
Samsung Galaxy Tab S8 with One UI 5.0 (Android 13)
Krita 5.1.2 and Krita Next (2022-12-10)
Comment 1 sh_zam 2022-12-23 14:01:23 UTC
Git commit 8d96b64dcdd6a7e1dcd7783a7866e0d208a48772 by Sharaf Zaman.
Committed on 23/12/2022 at 14:01.
Pushed by szaman into branch 'krita/5.1'.

Revert "Disable the touch docker on Android"

This reverts commit 56456f59f898536a470bcf19832788aa8bb0d6bb.

Not having touch docker caused this bug. I don't exactly know the reason
why this happened in the first place and given the difficulty associated
with bugs involving events pipeline, I think it is best to leave it in
the same state as it was before. But maybe fix the touch docker to
disable or delegate the buggy actions.
Related: bug 463318

M  +1    -1    plugins/dockers/CMakeLists.txt

https://invent.kde.org/graphics/krita/commit/8d96b64dcdd6a7e1dcd7783a7866e0d208a48772
Comment 2 sh_zam 2022-12-23 14:01:54 UTC
Git commit 41d9ad305c3a57d7565bba5eab5d8828a3fab609 by Sharaf Zaman.
Committed on 23/12/2022 at 14:01.
Pushed by szaman into branch 'master'.

Revert "Disable the touch docker on Android"

This reverts commit 56456f59f898536a470bcf19832788aa8bb0d6bb.

Not having touch docker caused this bug. I don't exactly know the reason
why this happened in the first place and given the difficulty associated
with bugs involving events pipeline, I think it is best to leave it in
the same state as it was before. But maybe fix the touch docker to
disable or delegate the buggy actions.
Related: bug 463318


(cherry picked from commit 8d96b64dcdd6a7e1dcd7783a7866e0d208a48772)

M  +1    -1    plugins/dockers/CMakeLists.txt

https://invent.kde.org/graphics/krita/commit/41d9ad305c3a57d7565bba5eab5d8828a3fab609
Comment 3 sh_zam 2022-12-23 14:03:13 UTC
Hello Tyson!

This probably needs to be fixed in a better way, but do back buttons/gesture not work in restoring Krita to the previous workable state?
Comment 4 Tyson Tan 2022-12-23 14:19:52 UTC
(In reply to sh_zam from comment #3)
> Hello Tyson!
> 
> This probably needs to be fixed in a better way, but do back buttons/gesture
> not work in restoring Krita to the previous workable state?

I'm not sure I understand what you said.

By "Back Buttons/Gestures", did you mean the "Revert" action?
Comment 5 sh_zam 2022-12-23 14:25:08 UTC
> By “Back Buttons/Gestures”, did you mean the “Revert” action?

I’m talking about the default Back Gesture that you perform OS wide to go back a screen (for Android smartphones it is swiping from Left/Right, something like this: <https://support.google.com/android/answer/9079644?hl=en#zippy=%2Cgo-back>)
Comment 6 Tyson Tan 2022-12-23 14:31:20 UTC
I never used system-wide Back button/gesture in Krita. What do you expect to happen?
Comment 7 sh_zam 2022-12-23 14:35:31 UTC
It successfully closes the white screen in my case (but I run Android 10 on my tablet), so things could be different.
Comment 8 Tyson Tan 2022-12-23 14:39:53 UTC
OK, I will check when the new build is up.
Comment 9 Tyson Tan 2022-12-24 09:37:45 UTC
The Back gesture was able to get me out of the white screen.