Differential D6850

KrViewer looses focus - fixed
ClosedPublic
Actions

Authored by miroslavm on Jul 23 2017, 12:47 PM.

Details

Reviewers
asensi
Group Reviewers
Krusader
Commits
R167:4f0fbcb507bb: KrViewer looses focus - fixed
Summary

FIXED: [363437] Edit field looses focus after closing one of Editor's/Viewer's tab
FIXED: [376372] KrViewer window loses focus on closing tabs BUG: 363437 BUG: 376382

Test Plan

Open a file with F4, open a second file with F4, close tab -> KrViewer remains focused with first tab

Diff Detail

Repository
R167 Krusader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
miroslavm created this revision.Jul 23 2017, 12:47 PM
miroslavm updated this revision to Diff 17055.Jul 23 2017, 12:56 PM

FIXED: [363437] Edit field looses focus after closing one of Editor's/Viewer's tab
FIXED: [376372] KrViewer window loses focus on closing tabs
BUG: 363437
BUG: 376382

miroslavm added a comment.EditedJul 23 2017, 12:59 PM

I have also added the changes of D6788 with arc in the previous diff ... :(

asensi accepted this revision.Aug 2 2017, 10:26 PM
asensi added a subscriber: asensi.

It can be compiled and executed using Kubuntu 17.04, and everything seems to work correctly :-)

This revision is now accepted and ready to land.Aug 2 2017, 10:26 PM
Closed by commit R167:4f0fbcb507bb: KrViewer looses focus - fixed (authored by miroslavm, committed by Alexander Bikadorov <alex.bikadorov@kdemail.net>). · Explain WhyAug 5 2017, 6:14 PM
This revision was automatically updated to reflect the committed changes.
abika added a subscriber: abika.Aug 5 2017, 6:16 PM

Sorry, toke me some time. I corrected a typo in one of the BUG: references. Thanks for the patch!

Revision Contents
Changeset List

PathPackages
Mkrusader/KViewer/krviewer.cpp (5 lines)

Diff 17759

View Options

krusader/KViewer/krviewer.cpp

Loading...