Bug 479440

Summary: Issue with undocked browser preview in onlinequoteseditor
Product: [Frameworks and Libraries] alkimia Reporter: Ralf Habacker <ralf.habacker>
Component: Online quotes widgetAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: 8.1.2   
Target Milestone: ---   
Platform: Other   
OS: All   
Latest Commit: Version Fixed In: 8.2
Sentry Crash Report:
Attachments: Screencast showing the issue

Description Ralf Habacker 2024-01-05 16:19:59 UTC
Created attachment 164691 [details]
Screencast showing the issue

SUMMARY
I tried the browser preview and I noticed an issue when the widget is undocked.

STEPS TO REPRODUCE
1. See this screencast

OBSERVED RESULT
I noticed that when the widget is docked, scrolling with the mousewheel and the scrollbar on the right works fine.
When you undock the browser window, only scrolling with the mouse works. As soon as I click on the scrollbar to move it, the entire content is lost.

EXPECTED RESULT
When undocking the browser window the content should not be lost.

SOFTWARE/OS VERSIONS
Linux:  openSUSE Leap 15.5
KDE Frameworks Version: 5.103
Qt Version: 5.15

ADDITIONAL INFORMATION
Thomas Baumgart reported this issue by email
Comment 1 Ralf Habacker 2024-01-06 09:35:42 UTC
The prerequisite for displaying this problem is that the build is configured with -DBUILD_WITH_WEBKIT or -DBUILD_WITH_WEBENGINE, which activates the field for editing the URL in the browser docking window.
Comment 2 Ralf Habacker 2024-01-14 18:38:29 UTC
Further investigations showed that this problem does not depend on a specific Qt version, as it occurs with Qt5 and an older version if the block is activated from line https://invent.kde.org/office/alkimia/-/blob/master/tools/onlinequoteseditor/mainwindow.cpp?ref_type=heads#L119 by defined(BUILD_WITH_WEBKIT) || defined(BUILD_WITH_WEBENGINE). It is caused by a mutual triggering of signals and their handling
Comment 3 Ralf Habacker 2024-03-27 22:35:07 UTC
Git commit 23203854264b5837939be9e7bb0bb4d19e9915df by Ralf Habacker.
Committed on 27/03/2024 at 18:00.
Pushed by habacker into branch 'master'.

Add download type to class AlkOnlineQuote[Source] and in ui

With this commit one instance of class AlkWebPage is used
for downloading and showing the html content, which fixed
bug 479440.
Related: bug 483138
FIXED-IN:8.2

M  +11   -7    autotests/alkonlinequotetest.cpp
M  +1    -1    autotests/alkonlinequotetest.h
M  +7    -5    src/alkonlinequote.cpp
M  +55   -54   src/alkonlinequote_p.cpp
M  +5    -5    src/alkonlinequote_p.h
M  +120  -111  src/alkonlinequotedetails.ui
M  +31   -1    src/alkonlinequotesource.cpp
M  +11   -2    src/alkonlinequotesource.h
M  +18   -17   src/alkonlinequotesource_p.h
M  +25   -1    src/alkonlinequotesprofile_p.cpp
M  +2    -0    src/alkonlinequotesprofile_p.h
M  +11   -0    src/alkonlinequoteswidget.cpp

https://invent.kde.org/office/alkimia/-/commit/23203854264b5837939be9e7bb0bb4d19e9915df