Bug 369308

Summary:	UX issues with the startup splash screen / about dialog
Product:	[Applications] krita	Reporter:	Alvin Wong <alvin>
Component:	General	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	scottpetrovic
Priority:	NOR
Version First Reported In:	git master (please specify the git hash!)
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	http://commits.kde.org/krita/89c0509824436c46ce2266ea267d23b7632cd3bf	Version Fixed/Implemented In:
Sentry Crash Report:

Description Alvin Wong 2016-09-25 08:02:51 UTC

1. When Krita is still loading, the splash screen is unresponsive and links and recent files are non-functional. On Windows, attempts at clicking on the window can cause the "unresponsive program" dialog to show, And if the user does attempt to click on the links, the clicks might still be registered after Krita has finished loading.

This appears to me as bad UX. "Unclickable" UI elements that appears to be clickable are bad, users may attempt to click on them but to receive no response, or even worse, a response after a long period.

The easy way would be to hide the links portions while Krita is still loading. Even though the splash screen would still be unresponsive, users would normally not interact with it and should not trigger the "unresponsive program" dialog.

2. The splash screen doesn't close when pressing `Esc`. I would expect users to expect this to work.

3. The about dialog uses the same UI widget as the splash screen, which shows the "Recent Files" links. Those seems weird to be on the about dialog, so I would suggest hiding them in the about dialog.

Reproducible: Always

Comment 1 Scott Petrovic 2016-09-28 01:59:00 UTC

This sounds like bugs in general. The links aren't clickable while Krita is starting up. Also clicking on the splash screen on startup causes Krita to be unresponsive and crash.

I would say we just need to disable any clicking behavior and hide any links while Krita is starting up. Hopefully that will prevent any crashes from happening.

Comment 2 Scott Petrovic 2016-10-21 02:03:58 UTC

Git commit 323f509f341fca03c4236c7397620846e7078a5c by Scott Petrovic.
Committed on 21/10/2016 at 02:03.
Pushed by scottpetrovic into branch 'petrovic/brush-editor-ui'.

M  +6    -0    libs/ui/KisApplication.cpp
M  +3    -1    libs/ui/dialogs/kis_about_application.cpp
M  +42   -24   libs/ui/kis_splash_screen.cpp
M  +2    -0    libs/ui/kis_splash_screen.h
M  +2    -2    libs/ui/wdgsplash.ui

http://commits.kde.org/krita/323f509f341fca03c4236c7397620846e7078a5c

Comment 3 Scott Petrovic 2016-10-21 02:15:58 UTC

with this commit, I am hiding the website links and recent files until everything is loaded. i think the splash screen's space will be collapsed in the meantime so there won't be any dead space. I also cleaned up the about us screen a bit with your recommendations.

Comment 4 Scott Petrovic 2016-10-23 20:12:24 UTC

Git commit 34b63f14beab5746568b5ea6ee47008ebaa473d8 by Scott Petrovic.
Committed on 23/10/2016 at 20:11.
Pushed by scottpetrovic into branch 'master'.

M  +6    -0    libs/ui/KisApplication.cpp
M  +3    -1    libs/ui/dialogs/kis_about_application.cpp
M  +42   -24   libs/ui/kis_splash_screen.cpp
M  +2    -0    libs/ui/kis_splash_screen.h
M  +2    -2    libs/ui/wdgsplash.ui

http://commits.kde.org/krita/34b63f14beab5746568b5ea6ee47008ebaa473d8

Comment 5 Scott Petrovic 2016-10-25 14:08:30 UTC

Git commit d970338a280ee139c4448957bf72b918997ed4a6 by Scott Petrovic.
Committed on 25/10/2016 at 14:06.
Pushed by scottpetrovic into branch 'petrovic/popup-palette-ui'.

M  +2    -1    libs/ui/KisApplication.cpp
M  +4    -0    libs/ui/dialogs/kis_about_application.cpp
M  +11   -7    libs/ui/kis_splash_screen.cpp
M  +2    -1    libs/ui/kis_splash_screen.h

http://commits.kde.org/krita/d970338a280ee139c4448957bf72b918997ed4a6

Comment 6 Scott Petrovic 2016-10-25 14:09:47 UTC

Git commit 89c0509824436c46ce2266ea267d23b7632cd3bf by Scott Petrovic.
Committed on 25/10/2016 at 14:09.
Pushed by scottpetrovic into branch 'master'.

M  +2    -1    libs/ui/KisApplication.cpp
M  +4    -0    libs/ui/dialogs/kis_about_application.cpp
M  +11   -7    libs/ui/kis_splash_screen.cpp
M  +2    -1    libs/ui/kis_splash_screen.h

http://commits.kde.org/krita/89c0509824436c46ce2266ea267d23b7632cd3bf

Comment 7 Scott Petrovic 2016-10-25 14:11:32 UTC

I added a commit so the links are shown to the website and docs on the About Us window. I think that is a good idea from boud.