Bug 440583 - The editor window becomes unusable in wayland
Summary: The editor window becomes unusable in wayland
Status: RESOLVED FIXED
Alias: None
Product: kate
Classification: Applications
Component: general (show other bugs)
Version: Git
Platform: Compiled Sources Linux
: NOR major
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-04 09:09 UTC by Jaime Torres
Modified: 2021-10-24 15:40 UTC (History)
3 users (show)

See Also:
Latest Commit: https://invent.kde.org/utilities/kate/commit/2fd3f8077f721c9b225ea37c7eff9aa670587753
Version Fixed In:
Sentry Crash Report:

Attachments
Strange behaviour (3.81 KB, image/png)
2021-08-04 09:11 UTC, Jaime Torres 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Jaime Torres 2021-08-04 09:09:08 UTC 
SUMMARY
The editor window becomes unusable

STEPS TO REPRODUCE
1. Open a text file
2. Have some lines in the text file
3. Modify the file outside kate
4. Once you press reload on the "Documents modified on Disk" dialog,
  kate becomes unusable.

OBSERVED RESULT
* some lines are only rendered with the upper half part of the font. 
* some lines are fixed in the bottom of the editor window
* I can't see the cursor any more.

EXPECTED RESULT
 It works as before the file is modified outside kate.

SOFTWARE/OS VERSIONS
KDE Frameworks 5.85.0
Qt 5.15.2 (built against 5.15.2)
The wayland windowing system
kate Version 21.11.70

ADDITIONAL INFORMATION
Comment 1 Jaime Torres 2021-08-04 09:11:41 UTC 
Created attachment 140504 [details]
Strange behaviour
Comment 2 cauchy 2021-09-02 06:15:18 UTC 
Operating System: Arch Linux (latest)
KDE Plasma Version: 5.22.5
KDE Frameworks Version: 5.85.0
Qt Version: 5.15.2
Kate 21.08
Full Wayland

+1 from me - exactly the same problem
Comment 3 Jaime Torres 2021-09-03 08:44:50 UTC 
A crash closing kate after reloading the view.

[KCrash Handler]
#6  0x000055ff3106e373 in qDeleteAll<__gnu_cxx::__normal_iterator<KateMDI::ToolView* const*, std::vector<KateMDI::ToolView*, std::allocator<KateMDI::ToolView*> > > > (end=..., begin=0x55ff33a223f0) at /usr/include/qt5/QtCore/qalgorithms.h:320
#7  qDeleteAll<std::vector<KateMDI::ToolView*, std::allocator<KateMDI::ToolView*> > > (c=std::vector of length 1, capacity 16 = {...}) at /usr/include/qt5/QtCore/qalgorithms.h:328
#8  KateMDI::MainWindow::~MainWindow (__vtt_parm=<optimized out>, this=<optimized out>, this=<optimized out>, __vtt_parm=<optimized out>) at /usr/src/debug/kate-21.08.0-1.1.x86_64/kate/katemdi.cpp:778
#9  KateMainWindow::~KateMainWindow (this=<optimized out>, this=<optimized out>) at /usr/src/debug/kate-21.08.0-1.1.x86_64/kate/katemainwindow.cpp:203
#10 0x000055ff31064340 in KateMainWindow::~KateMainWindow (this=<optimized out>, this=<optimized out>) at /usr/src/debug/kate-21.08.0-1.1.x86_64/kate/katemainwindow.cpp:181
#11 KateApp::~KateApp (this=<optimized out>, this=<optimized out>) at /usr/src/debug/kate-21.08.0-1.1.x86_64/kate/kateapp.cpp:113
#12 0x000055ff3105ded4 in main (argc=<optimized out>, argv=<optimized out>) at /usr/src/debug/kate-21.08.0-1.1.x86_64/kate/main.cpp:689
Comment 4 Christoph Cullmann 2021-10-24 14:58:00 UTC 
True, with the "Use a separate dialog for handling..." option enabled, all is in perfect chaos after some external change here, too.

Without that option set and just the inline "reload" messages all is fine.
Comment 5 Bug Janitor Service 2021-10-24 15:08:52 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/utilities/kate/-/merge_requests/504
Comment 6 Christoph Cullmann 2021-10-24 15:40:33 UTC 
Git commit 2fd3f8077f721c9b225ea37c7eff9aa670587753 by Christoph Cullmann.
Committed on 24/10/2021 at 15:05.
Pushed by cullmann into branch 'master'.

fixes reload dialog on wayland

avoid that after initial triggering this repaint and
more of main window is destroyed

M  +0    -2    kate/katemainwindow.cpp

https://invent.kde.org/utilities/kate/commit/2fd3f8077f721c9b225ea37c7eff9aa670587753