Removed unused signal for adjusting panel path label
Became obsolete with changing panel origin bar to KUrlNavigator.
Added icon to locked tabs
+ cleanup and style fixes.
martinkostolny |
Krusader |
Removed unused signal for adjusting panel path label
Became obsolete with changing panel origin bar to KUrlNavigator.
Added icon to locked tabs
+ cleanup and style fixes.
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
This is perfect! I also wanted to visualize the locked state but couldn't figure out how. This is a very elegant solution. Thanks!
I was testing with the D5744 already applied so I've got an issue with some profiles. When existing tab is locked and a the same tab is recreated from a profile (not locked), the "locked" icon stays. And vice versa. This (applied on top of your changes) should fix that:
diff --git a/krusader/panelmanager.cpp b/krusader/panelmanager.cpp index 19044158..ae97497f 100644 --- a/krusader/panelmanager.cpp +++ b/krusader/panelmanager.cpp @@ -206,6 +206,7 @@ void PanelManager::loadSettings(KConfigGroup config) ListPanel *panel = i < numTabsOld ? _tabbar->getPanel(i) : addPanel(false, grpTab); panel->restoreSettings(grpTab); + _tabbar->updateTab(panel); } for(int i = numTabsOld - 1; i >= numTabsNew && i > 0; i--)
Thanks Martin! Not a surprise I missed something again during testing:/
I added the line to the commit.