Bug 453989

Summary: With new DrKonqi dialogue, no way to get back from "Report Bug" to start
Product: [Applications] drkonqi Reporter: Jonathan Marten <jjm>
Component: generalAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: master   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Jonathan Marten 2022-05-18 15:07:10 UTC
SUMMARY

The first dialogue page displayed ("We are sorry, <program> closed unexpectedly") has buttons "Report Bug", "Restart Application" and "Developer Information".  If "Report Bug" is clicked on (possibly by mistake), there is no way to get back to the initial page to select another option - the "Back" button that normally appears there is missing.  The only way to get out is to close the dialogue, losing the crash information.

This does not happen if "Developer Information" is clicked, because the "Back" and "Forward" buttons are present at the top left.

May be related: when the dialogue is first shown there are no buttons at the top left.  Clicking on "Developer Information" shows the backtrace page with "Back" (enabled) and "Forward" (disabled) buttons at the top left.  Clicking "Back" returns to the first page but now with "Back" (disabled) and "Forward" (enabled) buttons present.

STEPS TO REPRODUCE
1.  After a program has crashed, click on the system tray icon or "Report Bug" on the notification.
2.  Click on "Report Bug".

OBSERVED RESULT

There should be a way to get back to the initial page

EXPECTED RESULT

There appears to be no way to get back

SOFTWARE/OS VERSIONS
KDE Plasma Version:  5.24.80
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.3

ADDITIONAL INFORMATION

There is also no margin around the "I Agree to be Contacted" button on the "Welcome to the Reporting Assistant" page.
Comment 1 Bug Janitor Service 2022-05-19 09:58:30 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/drkonqi/-/merge_requests/60
Comment 2 Harald Sitter 2022-05-22 21:20:33 UTC
Git commit d392dd6e6e1e543a7a0cea2d9ef0bd5521dd603f by Harald Sitter.
Committed on 22/05/2022 at 21:17.
Pushed by sitter into branch 'master'.

allow navigating back to the mainpage

this was not possible in the original implementation but we can now
easily allow this thanks to the pagestack, which will take care that the
page order is still reasonable

M  +1    -8    src/qml/MainPage.qml

https://invent.kde.org/plasma/drkonqi/commit/d392dd6e6e1e543a7a0cea2d9ef0bd5521dd603f
Comment 3 Harald Sitter 2022-05-22 21:22:47 UTC
Git commit 9a2c3c20e8fe3b710316537e108c00cdb6a50db7 by Harald Sitter.
Committed on 22/05/2022 at 21:22.
Pushed by sitter into branch 'Plasma/5.25'.

allow navigating back to the mainpage

this was not possible in the original implementation but we can now
easily allow this thanks to the pagestack, which will take care that the
page order is still reasonable


(cherry picked from commit d392dd6e6e1e543a7a0cea2d9ef0bd5521dd603f)

M  +1    -8    src/qml/MainPage.qml

https://invent.kde.org/plasma/drkonqi/commit/9a2c3c20e8fe3b710316537e108c00cdb6a50db7