diff options
| author | Robert Jennings <[email protected]> | 2018-04-25 12:35:29 +0200 |
|---|---|---|
| committer | Elvis Angelaccio <[email protected]> | 2018-04-25 12:37:28 +0200 |
| commit | 78c8b36dd84a94916835d702ea4a9e8eda65e8ef (patch) | |
| tree | de7a6f762d49f4149bd7b7062cad2a8ff8a73c2c /src | |
| parent | 74c2d4ea2d256183dc1f869a78ee5b1b79888330 (diff) | |
Update window title after closing split view
Summary:
In dolphintabpage.cpp:
Through the connections set up in createViewContainer(), m_primaryViewContainer->setActive(true), at line 98,
ends up triggering slotViewActivated().
Because m_primaryViewActive is true, oldActiveView is not set to the view needed in this particular situation.
Both oldActiveView and newActiveView end up pointing to the same view and the connections are not set up.
BUG: 385111
FIXED-IN: 18.04.1
Test Plan:
1. Open split view
2. Make left panel active
3. Close split view
4. Navigate to various directories
Window title now updates to show current directory
Also tried with multiple tabs/combinations
Reviewers: #dolphin, ngraham
Reviewed By: #dolphin, ngraham
Subscribers: elvisangelaccio, ngraham
Differential Revision: https://phabricator.kde.org/D12446
Diffstat (limited to 'src')
| -rw-r--r-- | src/dolphintabpage.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/dolphintabpage.cpp b/src/dolphintabpage.cpp index 342d6f6de..a96c8b6a3 100644 --- a/src/dolphintabpage.cpp +++ b/src/dolphintabpage.cpp @@ -94,6 +94,7 @@ void DolphinTabPage::setSplitViewEnabled(bool enabled, const QUrl &secondaryUrl) // If the primary view is active, we have to swap the pointers // because the secondary view will be the new primary view. qSwap(m_primaryViewContainer, m_secondaryViewContainer); + m_primaryViewActive = false; } m_primaryViewContainer->setActive(true); view->close(); |
